Unified Diff: lib/filterClasses.js
Issue 29855590:
Noissue - Avoid :-abp-properties() legacy syntax parsing for most filters (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created Aug. 14, 2018, 5:22 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
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 |
| @@ -122,17 +122,17 @@ |
| let filter = Filter.knownFilters.get(text); |
| if (filter) |
| return filter; |
| let match = text.includes("#") ? Filter.contentRegExp.exec(text) : null; |
| if (match) |
| { |
| let propsMatch; |
| - if (!match[2] && |
| + if (!match[2] && match[3].includes("[-") && |
| (propsMatch = /\[-abp-properties=(["'])([^"']+)\1\]/.exec(match[3]))) |
| { |
| // This is legacy CSS properties syntax, convert to current syntax |
| let prefix = match[3].substr(0, propsMatch.index); |
| let expression = propsMatch[2]; |
| let suffix = match[3].substr(propsMatch.index + propsMatch[0].length); |
| return Filter.fromText(`${match[1]}#?#` + |
| `${prefix}:-abp-properties(${expression})${suffix}`); |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
