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

Unified Diff: test/elemHide.js

Issue 29375915: Issue 4878 - Start using ESLint for adblockpluscore (Closed)
Patch Set: Created Feb. 20, 2017, 10:02 a.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
Index: test/elemHide.js
diff --git a/test/elemHide.js b/test/elemHide.js
index 93da5790bae3db70ff919cbcd51b9db68108385f..63e945ff441120779db04a648449d7b754ba32ef 100644
--- a/test/elemHide.js
+++ b/test/elemHide.js
@@ -37,9 +37,9 @@ function normalizeSelectors(selectors)
{
// getSelectorsForDomain is currently allowed to return duplicate selectors
// for performance reasons, so we need to remove duplicates here.
- return selectors.sort().filter((selector, index, selectors) =>
+ return selectors.sort().filter((selector, index, sortedSelectors) =>
{
- return index == 0 || selector != selectors[index - 1];
+ return index == 0 || selector != sortedSelectors[index - 1];
});
}
@@ -215,7 +215,7 @@ exports.testGetSelectorsForDomain = function(test)
testResult(test, "foo.com", [], ElemHide.SPECIFIC_ONLY);
// Note: We don't take care to track conditional selectors which became
// unconditional when a filter was removed. This was too expensive.
- //testResult(test, "foo.com", [], ElemHide.NO_UNCONDITIONAL);
+ // testResult(test, "foo.com", [], ElemHide.NO_UNCONDITIONAL);
testResult(test, "foo.com", ["hello"], ElemHide.ALL_MATCHING);
testResult(test, "foo.com", ["hello"]);
testResult(test, "bar.com", [], ElemHide.SPECIFIC_ONLY);

Powered by Google App Engine
This is Rietveld