Unified Diff: lib/filterClasses.js
Issue 29383960:
Issue 3143 - Filter elements with :-abp-has() (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore
Patch Set: Validate the syntax, remove unused flags
Created April 10, 2017, 1:12 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« chrome/content/elemHideEmulation.js ('K')
|
« chrome/content/elemHideEmulation.js ('k')
|
test/browser/elemHideEmulation.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 |
| @@ -983,17 +983,18 @@ |
| // Note: The ElemHide.prototype.domainSeparator is duplicated here, if that |
| // changes this must be changed too. |
| if (domain && /(^|,)~?(,|$)/.test(domain)) |
| return new InvalidFilter(text, "filter_invalid_domain"); |
| if (isException) |
| return new ElemHideException(text, domain, selector); |
| - if (selector.indexOf("[-abp-properties=") != -1) |
| + if ((selector.indexOf("[-abp-properties=") != -1) || |
| + (selector.indexOf("[-abp-selector=") != -1)) |
| { |
| // Element hiding emulation filters are inefficient so we need to make sure |
| // that they're only applied if they specify active domains |
| if (!/,[^~][^,.]*\.[^,]/.test("," + domain)) |
| return new InvalidFilter(text, "filter_elemhideemulation_nodomain"); |
| return new ElemHideEmulationFilter(text, domain, selector); |
| } |
« chrome/content/elemHideEmulation.js ('K')
|
« chrome/content/elemHideEmulation.js ('k')
|
test/browser/elemHideEmulation.js » ('j')
|
no next file with comments »
