Issue 3225 - Make Diagnostics compatible with the upcoming Adblock Plus 2.7 release

chrome/content/watcher.js

 /*
  * This Source Code is subject to the terms of the Mozilla Public License
  * version 2.0 (the "License"). You can obtain a copy of the License at
  * http://mozilla.org/MPL/2.0/.
  */
 
 const Cc = Components.classes;
 const Ci = Components.interfaces;
 const Cr = Components.results;
 const Cu = Components.utils;
 
 Cu.import("resource://gre/modules/Services.jsm");
 
 function abprequire(module)
 {
   let result = {};
   result.wrappedJSObject = result;
   Services.obs.notifyObservers(result, "adblockplus-require", module);
   return result.exports;
 }
 
 let {Policy} = abprequire("contentPolicy");
 let {Filter} = abprequire("filterClasses");
 
 let origShouldAllow = Policy.shouldAllow;
 if (!origShouldAllow)
   window.close();
 
-let currentData = null;

[Thomas Greiner, 2015/12/07 15:24:20]

`currentData` is no longer used across multiple functions - only in
`replacementShouldAllow` so we can make it a local variable now.

[Wladimir Palant, 2015/12/07 15:37:55]

Done.

 let processingQueue = [];
 let notifier = null;
 
 // Randomize URI to work around bug 719376
 let stringBundle = Services.strings.createBundle("chrome://abpwatcher/locale/global.properties?" + Math.random());
 
 let clipboardHelper = Cc["@mozilla.org/widget/clipboardhelper;1"].getService(Ci.nsIClipboardHelper);
 
 function init()
 {
@@ skipping 11 matching lines @@
   setInterval(processQueue, 200);
 }
 
 function E(id)
 {
   return document.getElementById(id);
 }
 
 function replacementShouldAllow({contentType, location, frames, isPrivate})
 {
-  let startTime = null;
-  try
-  {
-    currentData = {
-      type: contentType,
-      location: location,
-      frames: frames,
-      isPrivate: isPrivate
-    };
-    startTime = Date.now();
-  }
-  catch(e)
-  {
-    Cu.reportError(e);
-  }
-
+  let startTime = Date.now();
+  let currentData = {
+    type: contentType,
+    location: location,
+    frames: frames,
+    isPrivate: isPrivate
+  };
   let ret;
+
   try
   {
     ret = origShouldAllow.apply(this, arguments);
     return ret;
   }
   finally
   {
     if (startTime !== null)
     {
       currentData.processingTime = (Date.now() - startTime);
@@ skipping 449 matching lines @@
     for (let i = 0; i < this.observers.length; i++)
       if (this.observers[i] == observer)
         this.observers.splice(i--, 1);
   },
   notifyObservers: function(operation, entry)
   {
     for each (let observer in this.observers)
       observer(this, operation, entry);
   }
 };