[$csp4 adblockpluschrome] Issue 5241 - Add support for Content Security Policy filters

lib/requestBlocker.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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 50 matching lines @@
     return;
 
   for (let type in browser.webRequest.ResourceType)
     yield resourceTypes.get(browser.webRequest.ResourceType[type]) || "OTHER";
 
   // WEBSOCKET and WEBRTC get addressed through workarounds, even if the
   // webRequest API is lacking support to block these kind of requests.
   yield "WEBSOCKET";
   yield "WEBRTC";
 
-  // POPUP and ELEMHIDE filters aren't blocked on the request level but by other
-  // means. They don't have a corresponding value in webRequest.ResourceType.
+  // POPUP, CSP and ELEMHIDE filters aren't blocked on the request level but by

[Sebastian Noack, 2018/03/16 23:52:39]

Technically, the CSP filter is implemented on the request level. Perhaps, this
comment would be more acurrate and less confusing with following change, now:

  "blocked on the request level" => "mapped to resource types"

[kzar, 2018/03/19 14:41:30]

Done.

+  // other means. They don't have a corresponding value in
+  // webRequest.ResourceType.
   yield "POPUP";
   yield "ELEMHIDE";
+  yield "CSP";
 }());
 
 function onBeforeRequestAsync(page, url, type, docDomain,
                               thirdParty, sitekey,
                               specificOnly, filter)
 {
   if (filter)
     FilterNotifier.emit("filter.hitCount", filter, 0, 0, page);
 
   if (devtools)
@@ skipping 133 matching lines @@
     return false;
   }
 
   return ext.webRequest.onBeforeRequest._dispatch(
      new URL(msg.url),
      msg.requestType,
      sender.page,
      sender.frame
   ).includes(false);
 });