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

lib/contentPolicy.js

Index: lib/contentPolicy.js
===================================================================
--- a/lib/contentPolicy.js
+++ b/lib/contentPolicy.js
@@ -95,18 +95,18 @@ var Policy = exports.Policy =
     {
       Utils.styleService.unregisterSheet(collapseStyle, Ci.nsIStyleSheetService.USER_SHEET);
     });
   },
 
   /**
    * Checks whether a node should be blocked, hides it if necessary
    * @param contentType {String}
-   * @param location {String}
-   * @param fremes {Object[]}
+   * @param location {String} location of the request, filter key if contentType is ELEMHIDE
+   * @param frames {Object[]}
    * @param isPrivate {Boolean}  true if the request belongs to a private browsing window
    * @return {Object} An object containing properties block, collapse and hits
    *                  indicating how this request should be handled.
    */
   shouldAllow: function({contentType, location, frames, isPrivate})
   {
     let hits = [];
 
@@ -173,17 +173,17 @@ var Policy = exports.Policy =
 
         if (frame == testSitekeyFrame)
           [testSitekey, testSitekeyFrame] = getSitekey(frames.slice(i + 1));
       }
     }
 
     if (!match && contentType == "ELEMHIDE")
     {
-      match = location;
+      match = ElemHide.getFilterByKey(location);
       location = match.text.replace(/^.*?#/, '#');
 
       if (!match.isActiveOnDomain(docDomain))
         return response(true, false);
 
       let exception = ElemHide.getException(match, docDomain);
       if (exception)
       {