Emit updates to all listeners on pending tx updates

Previously the metamask controller only supported a single UI event listener, which wasn't useful for having a separate notification UI open at the same time.

Also reduced the notification's complexity down to a single method, which is heavily re-used.

Still has an outstanding bug where if the plugin ui dismisses the last tx, it does not close the notification popup.

1 files changed, 1 insertions, 15 deletions

diff --git a/app/scripts/lib/notifications.js b/app/scripts/lib/notifications.js
index 278c7674e..4c2aa91de 100644
--- a/app/scripts/lib/notifications.js
+++ b/app/scripts/lib/notifications.js
@@ -1,25 +1,11 @@
 const extension = require('./extension')
 
 const notifications = {
-  createUnlockRequestNotification: createUnlockRequestNotification,
-  createTxNotification: createTxNotification,
-  createMsgNotification: createMsgNotification,
+  show: showNotification,
 }
 module.exports = notifications
 window.METAMASK_NOTIFIER = notifications
 
-function createUnlockRequestNotification (opts) {
-  showNotification()
-}
-
-function createTxNotification (state) {
-  showNotification()
-}
-
-function createMsgNotification (state) {
-  showNotification()
-}
-
 function showNotification() {
   extension.windows.getAll({}, (windows) => {