Issue 3407 - Remove Policy.isWindowWhitelisted() method and its callers

chrome/content/ui/sendReport.js

Index: chrome/content/ui/sendReport.js
===================================================================
--- a/chrome/content/ui/sendReport.js
+++ b/chrome/content/ui/sendReport.js
@@ -913,17 +913,17 @@ var issuesDataSource =
     }
     else
       return false;
   },
 
   collectData: function(wnd, windowURI, callback)
   {
     this.contentWnd = wnd;
-    this.whitelistFilter = Policy.isWindowWhitelisted(wnd);
+    this.whitelistFilter = Policy.isWhitelisted(windowURI.spec);

[Thomas Greiner, 2016/01/05 15:12:24]

In line 591 we implement a fallback for when `windowURI` is `null` so it
probably makes sense to implement it here as well:

windowURI ? windowURI.spec : wnd.location.href

[Wladimir Palant, 2016/01/05 15:22:48]

No, we will rather have to fix line 591 as part of issue 3375 - the wnd
parameter is going away, we shouldn't access content windows directly. windowURI
parameter cannot actually be null, see UI.openReportDialog().

[Thomas Greiner, 2016/01/05 16:19:55]

Fine with me then.

 
     if (!this.whitelistFilter && this.isEnabled)
     {
       // Find disabled filters in active subscriptions matching any of the requests
       let disabledMatcher = new CombinedMatcher();
       for (let subscription of FilterStorage.subscriptions)
       {
         if (subscription.disabled)