Return Promise from `editRpc` thunk

`editRpc` returned a thunk that didn't return a Promise, despite doing
async work. It now returns a Promise.

In the one place where this is used, it didn't seem important to update
the callsite to block on this finishing. Only one call followed it in
the event handler, and it didn't seem to depend on this.

ui/app/store/actions.js

@@ -1461,24 +1461,29 @@ export function updateAndSetCustomRpc (newRpc, chainId, ticker = 'ETH', nickname
 }
 
 export function editRpc (oldRpc, newRpc, chainId, ticker = 'ETH', nickname, rpcPrefs) {
-  return (dispatch) => {
+  return async (dispatch) => {
     log.debug(`background.delRpcTarget: ${oldRpc}`)
-    background.delCustomRpc(oldRpc, (err) => {
+    try {
-      if (err) {
+      promisifiedBackground.delCustomRpc(oldRpc)
-        log.error(err)
+    } catch (error) {
-        return dispatch(displayWarning('Had a problem removing network!'))
+      log.error(error)
-      }
+      dispatch(displayWarning('Had a problem removing network!'))
-      dispatch(setSelectedToken())
+      return
-      background.updateAndSetCustomRpc(newRpc, chainId, ticker, nickname || newRpc, rpcPrefs, (err) => {
+    }
-        if (err) {
+
-          log.error(err)
+    dispatch(setSelectedToken())
-          return dispatch(displayWarning('Had a problem changing networks!'))
+
-        }
+    try {
-        dispatch({
+      await promisifiedBackground.updateAndSetCustomRpc(newRpc, chainId, ticker, nickname || newRpc, rpcPrefs)
-          type: actionConstants.SET_RPC_TARGET,
+    } catch (error) {
-          value: newRpc,
+      log.error(error)
-        })
+      dispatch(displayWarning('Had a problem changing networks!'))
-      })
+      return
+    }
+
+    dispatch({
+      type: actionConstants.SET_RPC_TARGET,
+      value: newRpc,
     })
   }
 }