Unified Diff: lib/elemHide.js
Issue 29775597:
Noissue - Use default parameters (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created May 9, 2018, 3:54 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/elemHide.js |
| =================================================================== |
| --- a/lib/elemHide.js |
| +++ b/lib/elemHide.js |
| @@ -237,22 +237,20 @@ |
| * on a particular host name. |
| * @param {string} domain |
| * @param {number} [criteria] |
| * One of the following: ElemHide.ALL_MATCHING, ElemHide.NO_UNCONDITIONAL or |
| * ElemHide.SPECIFIC_ONLY. |
| * @returns {string[]} |
| * List of selectors. |
| */ |
| - getSelectorsForDomain(domain, criteria) |
| + getSelectorsForDomain(domain, criteria = ElemHide.ALL_MATCHING) |
| { |
| let selectors = []; |
| - if (typeof criteria == "undefined") |
| - criteria = ElemHide.ALL_MATCHING; |
| if (criteria < ElemHide.NO_UNCONDITIONAL) |
| selectors = this.getUnconditionalSelectors(); |
| let specificOnly = (criteria >= ElemHide.SPECIFIC_ONLY); |
| let excluded = new Set(); |
| let currentDomain = domain ? domain.toUpperCase() : ""; |
| // This code is a performance hot-spot, which is why we've made certain |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
