From b5d7724150845f4d76d472c378dd0d4b953bce33 Mon Sep 17 00:00:00 2001
From: Brian Clifton <brian@clifton.me>
Date: Tue, 13 Jun 2017 23:32:25 -0700
Subject: [PATCH] Merge pull request #9318 from brave/ledger-balance-update

fix ledger balance check enable/disable
---
 app/browser/tabs.js                  |  9 +++++++++
 app/ledger.js                        | 10 +++++++++-
 app/renderer/components/main/main.js | 17 -----------------
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/app/browser/tabs.js b/app/browser/tabs.js
index 96a58441481..dc85f6376a8 100644
--- a/app/browser/tabs.js
+++ b/app/browser/tabs.js
@@ -179,6 +179,15 @@ const updateAboutDetails = (tab, tabValue) => {
   const autofillAddresses = appState.getIn(['autofill', 'addresses'])
   const versionInformation = appState.getIn(['about', 'brave', 'versionInformation'])
   const aboutDetails = tabValue.get('aboutDetails')
+  // TODO(bridiver) - convert this to an action
+  if (url === 'about:preferences#payments') {
+    tab.on('destroyed', () => {
+      process.emit(messages.LEDGER_PAYMENTS_PRESENT, tabValue.get('tabId'), false)
+    })
+    process.emit(messages.LEDGER_PAYMENTS_PRESENT, tabValue.get('tabId'), true)
+  } else {
+    process.emit(messages.LEDGER_PAYMENTS_PRESENT, tabValue.get('tabId'), false)
+  }
   if (location === 'about:preferences' || location === 'about:contributions' || location === aboutUrls.get('about:contributions')) {
     const ledgerData = ledgerInfo.merge(publisherInfo).merge(preferencesData)
     tab.send(messages.LEDGER_UPDATED, ledgerData.toJS())
diff --git a/app/ledger.js b/app/ledger.js
index d2b7a44dc20..1217cd24e2f 100644
--- a/app/ledger.js
+++ b/app/ledger.js
@@ -490,8 +490,16 @@ if (ipc) {
     boot()
   })
 
-  ipc.on(messages.LEDGER_PAYMENTS_PRESENT, (event, presentP) => {
+  let ledgerPaymentsPresent = {}
+  // TODO(bridiver) - convert this to an action
+  process.on(messages.LEDGER_PAYMENTS_PRESENT, (tabId, presentP) => {
     if (presentP) {
+      ledgerPaymentsPresent[tabId] = presentP
+    } else {
+      delete ledgerPaymentsPresent[tabId]
+    }
+
+    if (Object.keys(ledgerPaymentsPresent).length > 0 && getSetting(settings.PAYMENTS_ENABLED)) {
       if (!balanceTimeoutId) getBalance()
     } else if (balanceTimeoutId) {
       clearTimeout(balanceTimeoutId)
diff --git a/app/renderer/components/main/main.js b/app/renderer/components/main/main.js
index 2fb9fdedc32..38dddaad8ff 100644
--- a/app/renderer/components/main/main.js
+++ b/app/renderer/components/main/main.js
@@ -59,7 +59,6 @@ const shieldState = require('../../../common/state/shieldState')
 const siteSettingsState = require('../../../common/state/siteSettingsState')
 const menuBarState = require('../../../common/state/menuBarState')
 const windowState = require('../../../common/state/windowState')
-const windowStore = require('../../../../js/stores/windowStore')
 
 // Util
 const _ = require('underscore')
@@ -503,22 +502,6 @@ class Main extends ImmutableComponent {
       self.resetAltMenuProcessing()
       windowActions.onBlur(getCurrentWindowId())
     }
-
-    windowStore.addChangeListener(function () {
-      const paymentsEnabled = getSetting(settings.PAYMENTS_ENABLED)
-      if (paymentsEnabled) {
-        const windowState = self.props.windowState
-        const tabs = windowState && windowState.get('tabs')
-        if (tabs) {
-          try {
-            const presentP = tabs.some((tab) => {
-              return tab.get('location') === 'about:preferences#payments'
-            })
-            ipc.send(messages.LEDGER_PAYMENTS_PRESENT, presentP)
-          } catch (ex) { }
-        }
-      }
-    })
   }
 
   checkForTitleMode () {