Unified Diff: lib/requestBlocker.js
Issue 29680696:
[$csp4 adblockpluschrome] Issue 5241 - Add support for Content Security Policy filters (Closed)
Patch Set: Addressed Sebastian's feedback
Created March 19, 2018, 2:40 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/csp.js ('K')
|
« lib/devtools.js ('k')
|
metadata.chrome » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/requestBlocker.js |
| diff --git a/lib/requestBlocker.js b/lib/requestBlocker.js |
| index 56f6313cea25fe63ebfef2c5b0498a70f989117b..27814c434c7efd24f6fa816b95019079cbf8fcdd 100644 |
| --- a/lib/requestBlocker.js |
| +++ b/lib/requestBlocker.js |
| @@ -68,10 +68,10 @@ exports.filterTypes = new Set(function*() |
| 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 mapped to resource types. |
| yield "POPUP"; |
| yield "ELEMHIDE"; |
| + yield "CSP"; |
| }()); |
| function onBeforeRequestAsync(page, url, type, docDomain, |
« lib/csp.js ('K')
|
« lib/devtools.js ('k')
|
metadata.chrome » ('j')
|
no next file with comments »
