Unified Diff: lib/filterClasses.js
Issue 29746563:
Noissue - Trim spaces around element hiding emulation selectors (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Add tests for incorrect syntax
Created April 18, 2018, 12:41 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
test/filterClasses.js » ('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/filterClasses.js |
| =================================================================== |
| --- a/lib/filterClasses.js |
| +++ b/lib/filterClasses.js |
| @@ -183,17 +183,17 @@ |
| // Don't remove spaces inside comments |
| if (/^ *!/.test(text)) |
| return text.trim(); |
| // Special treatment for element hiding filters, right side is allowed to |
| // contain spaces |
| if (Filter.elemhideRegExp.test(text)) |
| { |
| - let [, domain, separator, selector] = /^(.*?)(#@?#?)(.*)$/.exec(text); |
| + let [, domain, separator, selector] = /^(.*?)(#[@?]?#?)(.*)$/.exec(text); |
| return domain.replace(/ +/g, "") + separator + selector.trim(); |
| } |
| // For most regexp filters we strip all spaces, but $csp filter options |
| // are allowed to contain single (non trailing) spaces. |
| let strippedText = text.replace(/ +/g, ""); |
| if (!strippedText.includes("$") || !/\bcsp=/i.test(strippedText)) |
| return strippedText; |
« no previous file with comments
|
« no previous file
|
test/filterClasses.js » ('j')
|
no next file with comments »
