Issue 3251 - Delegate processing of element hiding hits to shouldAllowAsync() so that hits show up

lib/child/elemHide.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 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 16 matching lines @@
   if (property && property.get)
     delete proto.Components;
 }
 catch (e)
 {
   Cu.reportError(e);
 }
 
 let {XPCOMUtils} = Cu.import("resource://gre/modules/XPCOMUtils.jsm", {});
 
-let {shouldAllow} = require("child/contentPolicy");
+let {shouldAllowAsync} = require("child/contentPolicy");
 let {Utils} = require("utils");
 
 // The allowXBL binding below won't have any effect on the element. For elements
 // that should be hidden however we don't return any binding at all, this makes
 // Gecko stop constructing the node - it cannot be shown.
 const allowXBL = "<bindings xmlns='http://www.mozilla.org/xbl'><binding id='dummy' bindToUntrustedContent='true'/></bindings>";
 const hideXBL = "<bindings xmlns='http://www.mozilla.org/xbl'/>";
 
 /**
  * about: URL module used to count hits.
@@ skipping 76 matching lines @@
   notificationCallbacks: null,
   loadFlags: 0,
   loadGroup: null,
   name: null,
   status: Cr.NS_OK,
 
   asyncOpen: function(listener, context)
   {
     let processResponse = (allow) =>
     {
       let data = (allow ? allowXBL : hideXBL);

[tschuster, 2015/11/18 13:47:52]

Oh yeah. That is pretty subtle. Especially with the previous name.

       let stream = Cc["@mozilla.org/io/string-input-stream;1"].createInstance(Ci.nsIStringInputStream);
       stream.setData(data, data.length);
 
       try {
         listener.onStartRequest(this, context);
       } catch(e) {}
       try {
         listener.onDataAvailable(this, context, stream, 0, stream.available());
       } catch(e) {}
       try {
         listener.onStopRequest(this, context, Cr.NS_OK);
       } catch(e) {}
     };
 
     let window = Utils.getRequestWindow(this);
-    shouldAllow(window, window.document, "ELEMHIDE", this.key, processResponse);
+    shouldAllowAsync(window, window.document, "ELEMHIDE", this.key, processResponse);
   },
 
   asyncOpen2: function(listener)
   {
     if (!this.loadInfo.triggeringPrincipal.equals(Utils.systemPrincipal))
       throw Cr.NS_ERROR_FAILURE;
     this.asyncOpen(listener, null);
   },
 
   open: function()
@@ skipping 12 matching lines @@
   {
     throw Cr.NS_ERROR_NOT_IMPLEMENTED;
   },
   resume: function()
   {
     throw Cr.NS_ERROR_NOT_IMPLEMENTED;
   },
 
   QueryInterface: XPCOMUtils.generateQI([Ci.nsIChannel, Ci.nsIRequest])
 };