Unified Diff: lib/snippets.js
Issue 29865583:
Issue 6886 - Return snippet filters instead of just scripts (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: rename it getFiltersForDomain (for real)
Created Aug. 27, 2018, 12:21 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/snippets.js » ('j')
|
test/snippets.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/snippets.js |
| =================================================================== |
| --- a/lib/snippets.js |
| +++ b/lib/snippets.js |
| @@ -56,28 +56,28 @@ |
| * @param {SnippetFilter} filter |
| */ |
| remove(filter) |
| { |
| filters.delete(filter.text); |
| }, |
| /** |
| - * Returns a list of all scripts active on a particular domain |
| + * Returns a list of all snippet filters active on a particular domain |
| * @param {string} domain |
| - * @return {string[]} |
| + * @return {Array.<SnippetFilter>} |
| */ |
| - getScriptsForDomain(domain) |
| + getFiltersForDomain(domain) |
| { |
| let result = []; |
| for (let text of filters) |
| { |
| let filter = Filter.fromText(text); |
| if (filter.isActiveOnDomain(domain)) |
| - result.push(filter.script); |
| + result.push(filter); |
| } |
| return result; |
| } |
| }; |
| exports.Snippets = Snippets; |
| /** |
« no previous file with comments
|
« no previous file
|
test/snippets.js » ('j')
|
test/snippets.js » ('J')
