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

Unified Diff: test/elemHide.js

Issue 29342830: Issue 4057 - Further speedup ElemHide.getSelectorsforDomain (Closed)
Patch Set: Rebased, unconditionalSelectors Created May 24, 2016, 2:39 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
« lib/elemHide.js ('K') | « lib/elemHide.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/elemHide.js
diff --git a/test/elemHide.js b/test/elemHide.js
index f073108566b4769604ec3c77a1a5b6027e3dab15..b93baa4e4400ec3a923fb6ad515e874206128154 100644
--- a/test/elemHide.js
+++ b/test/elemHide.js
@@ -29,12 +29,8 @@ let {Filter} = require("filterClasses");
exports.testGetSelectorsForDomain = function(test)
{
- function fromText(f)
- {
- return (filterText) => f(Filter.fromText(filterText));
- }
- let addFilter = fromText(ElemHide.add);
- let removeFilter = fromText(ElemHide.remove);
+ let addFilter = filterText => ElemHide.add(Filter.fromText(filterText));
kzar 2016/05/24 14:45:22 `this` was being clobbered for the ElemHide method
+ let removeFilter = filterText => ElemHide.remove(Filter.fromText(filterText));
function normalizeSelectors(selectors)
{
« lib/elemHide.js ('K') | « lib/elemHide.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld