Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: test/filterListener.js

Issue 29935564: Issue 7452 - Do not cache element hiding filter objects by default Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Rename filter* to filterText* Created April 8, 2019, 9:59 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/elemHide.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/filterListener.js
===================================================================
--- a/test/filterListener.js
+++ b/test/filterListener.js
@@ -25,17 +25,17 @@
let Filter = null;
let defaultMatcher = null;
let SpecialSubscription = null;
exports.setUp = function(callback)
{
sandboxedRequire = createSandbox({
extraExports: {
- elemHide: ["knownFilters"],
+ elemHide: ["knownFilterText"],
elemHideEmulation: ["filters"],
elemHideExceptions: ["knownExceptions"]
}
});
// We need to require the filterListener module so that filter changes will be
// noticed, even though we don't directly use the module here.
sandboxedRequire("../lib/filterListener");
@@ -78,18 +78,18 @@
}
}
}
result[type] = filters;
}
let elemHide = sandboxedRequire("../lib/elemHide");
result.elemhide = [];
- for (let filter of elemHide.knownFilters)
- result.elemhide.push(filter.text);
+ for (let filterText of elemHide.knownFilterText)
+ result.elemhide.push(filterText);
let elemHideExceptions = sandboxedRequire("../lib/elemHideExceptions");
result.elemhideexception = [];
for (let exception of elemHideExceptions.knownExceptions)
result.elemhideexception.push(exception.text);
let elemHideEmulation = sandboxedRequire("../lib/elemHideEmulation");
result.elemhideemulation = [];
« no previous file with comments | « test/elemHide.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld