Fix inpage.js event handler memory leak (#6177)

Fixes #6176

1 files changed, 5 insertions, 4 deletions

diff --git a/app/scripts/inpage.js b/app/scripts/inpage.js
index fb2a84d1a..ae5a375b0 100644
--- a/app/scripts/inpage.js
+++ b/app/scripts/inpage.js
@@ -29,12 +29,13 @@ console.warn('ATTENTION: In an effort to improve user privacy, MetaMask ' +
  * @param {Function} handler - event handler
  * @param {boolean} remove - removes this handler after being triggered
  */
-function onMessage (messageType, handler, remove) {
-  window.addEventListener('message', function ({ data }) {
+function onMessage (messageType, callback, remove) {
+  const handler = function ({ data }) {
     if (!data || data.type !== messageType) { return }
     remove && window.removeEventListener('message', handler)
-    handler.apply(window, arguments)
-  })
+    callback.apply(window, arguments)
+  }
+  window.addEventListener('message', handler)
 }
 
 //