From 4e2923127515ff820eb41381dd25f6d2aa5a0584 Mon Sep 17 00:00:00 2001 From: "J. Ryan Stinnett" Date: Wed, 3 Aug 2016 18:13:52 -0500 Subject: [PATCH] Bug 1240912 - Beacon filtering no longer needed. r=ochameau MozReview-Commit-ID: EQ4HrvGFBiq --- devtools/shared/webconsole/network-monitor.js | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/devtools/shared/webconsole/network-monitor.js b/devtools/shared/webconsole/network-monitor.js index 41143fb9dd88..0998387f61db 100644 --- a/devtools/shared/webconsole/network-monitor.js +++ b/devtools/shared/webconsole/network-monitor.js @@ -112,25 +112,6 @@ function matchRequest(channel, filters) { } } - // The following check is necessary because beacon channels don't come - // associated with a load group. Bug 1160837 will hopefully introduce a - // platform fix that will render the following code entirely useless. - if (channel.loadInfo && - channel.loadInfo.externalContentPolicyType == - Ci.nsIContentPolicy.TYPE_BEACON) { - let nonE10sMatch = filters.window && - channel.loadInfo.loadingDocument === filters.window.document; - const loadingPrincipal = channel.loadInfo.loadingPrincipal; - let e10sMatch = filters.topFrame && - filters.topFrame.contentPrincipal && - filters.topFrame.contentPrincipal.equals(loadingPrincipal) && - filters.topFrame.contentPrincipal.URI.spec == channel.referrer.spec; - let b2gMatch = filters.appId && loadingPrincipal.appId === filters.appId; - if (nonE10sMatch || e10sMatch || b2gMatch) { - return true; - } - } - return false; }