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

Unified Diff: chrome/content/tests/test_requestList.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
« no previous file with comments | « chrome/content/tests/synchronizer.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/tests/test_requestList.html
===================================================================
--- a/chrome/content/tests/test_requestList.html
+++ b/chrome/content/tests/test_requestList.html
@@ -110,17 +110,17 @@
checkScanDone();
});
function checkScanDone()
{
if (wndScanComplete && frameScanComplete)
{
// Fold duplicate entries
- for each (let list in [wndData, frameData])
+ for (let list of [wndData, frameData])
{
let keys = {};
for (let i = 0; i < list.length; i++)
{
let key = " " + list[i].entry.location + " " + list[i].entry.type + " " + list[i].entry.docDomain;
if (key in keys)
{
list.splice(keys[key], 1);
« no previous file with comments | « chrome/content/tests/synchronizer.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld