only send ipc messages to the hostWebContents of the tab that generat…

…ed them fix #9701
brave · Jun 25, 2017 · bb82dfd · bb82dfd
1 parent d99c5c6
commit bb82dfd
Showing 1 changed file with 20 additions and 10 deletions.
diff --git a/app/filtering.js b/app/filtering.js
@@ -137,11 +137,16 @@ function registerForBeforeRequest (session, partition) {
           appActions.addResourceCount(parentResourceName, 1)
         }
 
-        BrowserWindow.getAllWindows().forEach((wnd) =>
-          wnd.webContents.send(message, parentResourceName, {
-            tabId: details.tabId,
-            url: details.url
-          }))
+        // TODO(bridiver) - convert to appActions/appState
+        setImmediate(() => {
+          const tab = webContents.fromTabID(details.tabId)
+          if (tab && tab.hostWebContents) {
+            tab.hostWebContents.send(message, parentResourceName, {
+              tabId: details.tabId,
+              url: details.url
+            })
+          }
+        })
 
         if (parentResourceName === appConfig.resourceNames.SAFE_BROWSING) {
           let redirectURL = appUrlUtil.getTargetAboutUrl('about:safebrowsing#' + details.url)
@@ -167,11 +172,16 @@ function registerForBeforeRequest (session, partition) {
           if (isHttpsEverywhere) {
             appActions.addResourceCount(results.resourceName, 1)
           }
-          BrowserWindow.getAllWindows().forEach((wnd) =>
-            wnd.webContents.send(messages.HTTPSE_RULE_APPLIED, results.ruleset, {
-              tabId: details.tabId,
-              url: details.url
-            }))
+          // TODO(bridiver) - convert to appActions/appState
+          setImmediate(() => {
+            const tab = webContents.fromTabID(details.tabId)
+            if (tab && tab.hostWebContents) {
+              tab.hostWebContents.send(messages.HTTPSE_RULE_APPLIED, results.ruleset, {
+                tabId: details.tabId,
+                url: details.url
+              })
+            }
+          })
         }
         cb({redirectURL: results.redirectURL})
         return