Issue 4162 - Rename about:abp-elemhidehit into about:abp-elemhide

lib/child/elemHide.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 36 matching lines @@
 const notImplemented = () => Cr.NS_ERROR_NOT_IMPLEMENTED;
 
 /**
  * about: URL module used to count hits.
  * @class
  */
 let AboutHandler =
 {
   classID: Components.ID("{55fb7be0-1dd2-11b2-98e6-9e97caf8ba67}"),
   classDescription: "Element hiding hit registration protocol handler",
-  aboutPrefix: "abp-elemhidehit",
+  aboutPrefix: "abp-elemhide",
 
   /**
    * Registers handler on startup.
    */
   init: function()
   {
     let registrar = Components.manager.QueryInterface(Ci.nsIComponentRegistrar);
     registrar.registerFactory(this.classID, this.classDescription,
         "@mozilla.org/network/protocol/about;1?what=" + this.aboutPrefix, this);
     onShutdown.add(function()
@@ skipping 18 matching lines @@
   // About module implementation
   //
 
   getURIFlags: function(uri)
   {
     return Ci.nsIAboutModule.HIDE_FROM_ABOUTABOUT;
   },
 
   newChannel: function(uri, loadInfo)
   {
-    let match = /\?(\d+|css)$/.exec(uri.path);
+    let match = /\?(?:hit(\d+)|css)$/.exec(uri.path);
     if (!match)
       throw Cr.NS_ERROR_FAILURE;
 
-    if (match[1] == "css")
+    if (match[1])
+      return new HitRegistrationChannel(uri, loadInfo, match[1]);
+    else
       return new StyleDataChannel(uri, loadInfo);
-    else
-      return new HitRegistrationChannel(uri, loadInfo, match[1]);
   },
 
   QueryInterface: XPCOMUtils.generateQI([Ci.nsIFactory, Ci.nsIAboutModule])
 };
 AboutHandler.init();
 
 /**
  * Base class for channel implementations, subclasses usually only need to
  * override BaseChannel._getResponse() method.
  * @constructor
@@ skipping 110 matching lines @@
   {
     function escapeChar(match)
     {
       return "\\" + match.charCodeAt(0).toString(16) + " ";
     }
 
     // Would be great to avoid sync messaging here but nsIStyleSheetService
     // insists on opening channels synchronously.
     let domains = port.emitSync("getSelectors");
 
-    let cssPrefix = "{-moz-binding: url(about:abp-elemhidehit?";
+    let cssPrefix = "{-moz-binding: url(about:abp-elemhide?hit";
     let cssSuffix = "#dummy) !important;}\n";
     let result = [];
 
     for (let [domain, selectors] of domains)
     {
       if (domain)
       {
         result.push('@-moz-document domain("',
             domain.replace(/[^\x01-\x7F]/g, escapeChar)
                   .split(",").join('"),domain("'),
@@ skipping 41 matching lines @@
     return new Promise((resolve, reject) =>
     {
       let window = Utils.getRequestWindow(this);
       shouldAllowAsync(window, window.document, "ELEMHIDE", this.key, allow =>
       {
         resolve(allow ? allowXBL : hideXBL);
       });
     });
   }
 };