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

Unified Diff: chrome/content/tests/performance/matching.html

Issue 6423769060999168: Issue 301 - adblockplustests: Use for (.. of ..) (Closed)
Patch Set: Created April 12, 2014, 1:48 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
Index: chrome/content/tests/performance/matching.html
===================================================================
--- a/chrome/content/tests/performance/matching.html
+++ b/chrome/content/tests/performance/matching.html
@@ -54,25 +54,25 @@
.join("\n");
addresses = document.getElementById("addresses").value.replace(/^[\r\n]+/, "")
.replace(/[\r\n]+$/, "")
.split(/[\r\n]+/);
document.getElementById("result").textContent = "";
document.getElementById("progress").style.display = "";
defaultMatcher.clear();
- for each (let filter in filters)
+ for (let filter of filters)
defaultMatcher.add(filter);
runTests(runTest, cleanup, finalize);
}
function runTest()
{
- for each (let address in addresses)
+ for (let address of addresses)
defaultMatcher.matchesAny(address, "IMAGE", null, false);
}
function cleanup()
{
// Add and remove a dummy filter to clear matcher's cache
let dummy = Filter.fromText("foobar");
defaultMatcher.add(dummy);
« no previous file with comments | « chrome/content/tests/performance/matcher_init.html ('k') | chrome/content/tests/performance/page_load_overhead.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld