Unified Diff: lib/common.js
Issue 30012569:
Issue 7291 - Don't mangle selectors with a '*' (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created Feb. 20, 2019, 8: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
|
test/common.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/common.js |
| =================================================================== |
| --- a/lib/common.js |
| +++ b/lib/common.js |
| @@ -201,16 +201,18 @@ |
| qualifiedSelector += ", "; |
| let index = findTargetSelectorIndex(sub); |
| let [, type = "", rest] = /^([a-z][a-z-]*)?(.*)/i.exec(sub.substr(index)); |
| // Note that the first group in the regular expression is optional. If it |
| // doesn't match (e.g. "#foo::nth-child(1)"), type will be an empty string. |
| - qualifiedSelector += sub.substr(0, index) + type + qualifier + rest; |
| + // If `rest` is "*" then it is discarded. |
| + qualifiedSelector += sub.substring(0, index) + type + qualifier + |
| + (rest != "*" ? rest : ""); |
| } |
| // Remove the initial comma and space. |
| return qualifiedSelector.substr(2); |
| } |
| exports.qualifySelector = qualifySelector; |
« no previous file with comments
|
« no previous file
|
test/common.js » ('j')
|
no next file with comments »
