Issue 2738 - Pass bit masks to matching functions

background.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 316 matching lines @@
 
       sendResponse(selectors);
       break;
     case "should-collapse":
       if (isFrameWhitelisted(sender.page, sender.frame, RegExpFilter.typeMap.DOCUMENT))
       {
         sendResponse(false);
         break;
       }
 
+      var typeMask = RegExpFilter.typeMap[msg.mediatype];
       var documentHost = extractHostFromFrame(sender.frame);
       var blocked = false;
-      var typeMask = RegExpFilter.typeMap[msg.mediatype];

[Sebastian Noack, 2015/07/13 16:41:50]

Nit: I'd put it above the declaration of documentHost. Then you have all
variables declared in the order they are used, plus the variables passed to the
matcher will be declared in the same place.

[kzar, 2015/07/14 09:22:23]

Done.

 
       for (var i = 0; i < msg.urls.length; i++)
       {
         var url = new URL(msg.urls[i], msg.baseURL);
         var filter = defaultMatcher.matchesAny(
           stringifyURL(url), typeMask,
           documentHost, isThirdParty(url, documentHost)
         );
 
         if (filter instanceof BlockingFilter)
@@ skipping 74 matching lines @@
       break;
   }
 });
 
 // update icon when page changes location
 ext.pages.onLoading.addListener(function(page)
 {
   page.sendMessage({type: "clickhide-deactivate"});
   refreshIconAndContextMenu(page);
 });