From 21d62e3adc222401bd7ab0a9c88073d82f221433 Mon Sep 17 00:00:00 2001
From: Mark Stacey <markjstacey@gmail.com>
Date: Mon, 27 Apr 2020 18:21:17 -0300
Subject: [PATCH] Use `async/await` for `forceUpdateMetamaskState` (#8429)

The `forceUpdateMetamaskState` function now uses `async/await` instead
of a Promise constructor. This was done to make an upcoming change
easier (making `updateMetamaskState` async).
---
 ui/app/store/actions.js | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/ui/app/store/actions.js b/ui/app/store/actions.js
index 920ab3b5b..816577930 100644
--- a/ui/app/store/actions.js
+++ b/ui/app/store/actions.js
@@ -1865,19 +1865,19 @@ export function setMouseUserState (isMouseUser) {
   }
 }
 
-export function forceUpdateMetamaskState (dispatch) {
+export async function forceUpdateMetamaskState (dispatch) {
   log.debug(`background.getState`)
-  return new Promise((resolve, reject) => {
-    background.getState((err, newState) => {
-      if (err) {
-        dispatch(displayWarning(err.message))
-        return reject(err)
-      }
 
-      dispatch(updateMetamaskState(newState))
-      resolve(newState)
-    })
-  })
+  let newState
+  try {
+    newState = await promisifiedBackground.getState()
+  } catch (error) {
+    dispatch(displayWarning(error.message))
+    throw error
+  }
+
+  dispatch(updateMetamaskState(newState))
+  return newState
 }
 
 export function toggleAccountMenu () {