From eea77b828fdaecb07fdc704edbcac0f42c771d27 Mon Sep 17 00:00:00 2001 From: Frankie Date: Wed, 14 Sep 2016 10:59:31 -0700 Subject: Dont generate a popup notification when submiting a transaction from with in MetaMask --- app/scripts/background.js | 28 +++++++++++----------------- 1 file changed, 11 insertions(+), 17 deletions(-) (limited to 'app') diff --git a/app/scripts/background.js b/app/scripts/background.js index 58228a41a..18a112dd1 100644 --- a/app/scripts/background.js +++ b/app/scripts/background.js @@ -10,31 +10,22 @@ const MetamaskController = require('./metamask-controller') const extension = require('./lib/extension') const STORAGE_KEY = 'metamask-config' - +var popupIsOpen = false const controller = new MetamaskController({ // User confirmation callbacks: - showUnconfirmedMessage, - unlockAccountMessage, - showUnconfirmedTx, + showUnconfirmedMessage: triggerUi, + unlockAccountMessage: triggerUi, + showUnconfirmedTx: triggerUi, // Persistence Methods: setData, loadData, }) const idStore = controller.idStore -function unlockAccountMessage () { - notification.show() -} - -function showUnconfirmedMessage (msgParams, msgId) { - notification.show() +function triggerUi () { + if (!popupIsOpen) notification.show() } - -function showUnconfirmedTx (txParams, txData, onTxDoneCb) { - notification.show() -} - // On first install, open a window to MetaMask website to how-it-works. extension.runtime.onInstalled.addListener(function (details) { @@ -53,7 +44,8 @@ function connectRemote (remotePort) { var portStream = new PortStream(remotePort) if (isMetaMaskInternalProcess) { // communication with popup - setupTrustedCommunication(portStream, 'MetaMask') + remotePort.name === 'popup' ? popupIsOpen = true : popupIsOpen = false + setupTrustedCommunication(portStream, 'MetaMask', remotePort.name) } else { // communication with page var originDomain = urlUtil.parse(remotePort.sender.url).hostname @@ -69,12 +61,13 @@ function setupUntrustedCommunication (connectionStream, originDomain) { controller.setupPublicConfig(mx.createStream('publicConfig')) } -function setupTrustedCommunication (connectionStream, originDomain) { +function setupTrustedCommunication (connectionStream, originDomain, metamaskContext) { // setup multiplexing var mx = setupMultiplex(connectionStream) // connect features setupControllerConnection(mx.createStream('controller')) controller.setupProviderConnection(mx.createStream('provider'), originDomain) + if (metamaskContext === 'popup') popupIsOpen = true } // @@ -95,6 +88,7 @@ function setupControllerConnection (stream) { // teardown on disconnect eos(stream, () => { controller.ethStore.removeListener('update', controller.sendUpdate.bind(controller)) + popupIsOpen = false }) }) } -- cgit v1.2.3 From 03094091d1ae1573f7867c887d91346356d48b6c Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Wed, 14 Sep 2016 11:35:50 -0700 Subject: Version 2.11.2 --- app/manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app') diff --git a/app/manifest.json b/app/manifest.json index a21e0612a..c1cbd074d 100644 --- a/app/manifest.json +++ b/app/manifest.json @@ -1,7 +1,7 @@ { "name": "MetaMask", "short_name": "Metamask", - "version": "2.11.1", + "version": "2.11.2", "manifest_version": 2, "author": "https://metamask.io", "description": "Ethereum Browser Extension", -- cgit v1.2.3 From 19a354066b66c33dd5f2f1c52f94bea4525cd2d1 Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Wed, 14 Sep 2016 11:38:59 -0700 Subject: Version 2.12.0 --- app/manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app') diff --git a/app/manifest.json b/app/manifest.json index c1cbd074d..fe4ecf4f0 100644 --- a/app/manifest.json +++ b/app/manifest.json @@ -1,7 +1,7 @@ { "name": "MetaMask", "short_name": "Metamask", - "version": "2.11.2", + "version": "2.12.0", "manifest_version": 2, "author": "https://metamask.io", "description": "Ethereum Browser Extension", -- cgit v1.2.3 From b7e6ec5ceae4510013c644bac7bc4d138aa6f0a1 Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Wed, 14 Sep 2016 11:57:41 -0700 Subject: Fix popup flag assignment --- app/scripts/background.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'app') diff --git a/app/scripts/background.js b/app/scripts/background.js index 18a112dd1..b2268425a 100644 --- a/app/scripts/background.js +++ b/app/scripts/background.js @@ -44,7 +44,7 @@ function connectRemote (remotePort) { var portStream = new PortStream(remotePort) if (isMetaMaskInternalProcess) { // communication with popup - remotePort.name === 'popup' ? popupIsOpen = true : popupIsOpen = false + popupIsOpen = remotePort.name === 'popup' setupTrustedCommunication(portStream, 'MetaMask', remotePort.name) } else { // communication with page @@ -67,7 +67,6 @@ function setupTrustedCommunication (connectionStream, originDomain, metamaskCont // connect features setupControllerConnection(mx.createStream('controller')) controller.setupProviderConnection(mx.createStream('provider'), originDomain) - if (metamaskContext === 'popup') popupIsOpen = true } // -- cgit v1.2.3 From 8e7a9c470943b4cfe1e19f1682ee8521033ed2db Mon Sep 17 00:00:00 2001 From: Frankie Date: Wed, 14 Sep 2016 12:22:39 -0700 Subject: Remove unused argument --- app/scripts/background.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app') diff --git a/app/scripts/background.js b/app/scripts/background.js index b2268425a..652acc113 100644 --- a/app/scripts/background.js +++ b/app/scripts/background.js @@ -61,7 +61,7 @@ function setupUntrustedCommunication (connectionStream, originDomain) { controller.setupPublicConfig(mx.createStream('publicConfig')) } -function setupTrustedCommunication (connectionStream, originDomain, metamaskContext) { +function setupTrustedCommunication (connectionStream, originDomain) { // setup multiplexing var mx = setupMultiplex(connectionStream) // connect features -- cgit v1.2.3 From 8df005153befe65cb605175254f37d7207acc9c5 Mon Sep 17 00:00:00 2001 From: Dan Finlay Date: Wed, 14 Sep 2016 15:51:50 -0700 Subject: Version 2.12.1 --- app/manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app') diff --git a/app/manifest.json b/app/manifest.json index fe4ecf4f0..daf9ef94a 100644 --- a/app/manifest.json +++ b/app/manifest.json @@ -1,7 +1,7 @@ { "name": "MetaMask", "short_name": "Metamask", - "version": "2.12.0", + "version": "2.12.1", "manifest_version": 2, "author": "https://metamask.io", "description": "Ethereum Browser Extension", -- cgit v1.2.3