Bug 1308998 - remove leaked i variable in browser-test.js; r=mccr8, a=test-only
authorJulian Descottes <jdescottes@mozilla.com>
Mon, 10 Oct 2016 20:18:51 +0200
changeset 350640 a34876e25e31baf3044ff0ed6144fa1e04a50494
parent 350639 c1eb14f02fc28acb10cdc50c0053c72a87921dca
child 350641 1116e0538d0059a0c00662c66a24357e7fe17708
push id1230
push userjlund@mozilla.com
push dateMon, 31 Oct 2016 18:13:35 +0000
treeherdermozilla-release@5e06e3766db2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmccr8, test-only
bugs1308998
milestone50.0
Bug 1308998 - remove leaked i variable in browser-test.js; r=mccr8, a=test-only MozReview-Commit-ID: Ej9KOLvoYLC
testing/mochitest/browser-test.js
--- a/testing/mochitest/browser-test.js
+++ b/testing/mochitest/browser-test.js
@@ -617,17 +617,17 @@ Tester.prototype = {
         // use a shrinking GC so that the JS engine will discard JIT code and
         // JIT caches more aggressively.
 
         let shutdownCleanup = aCallback => {
           Cu.schedulePreciseShrinkingGC(() => {
             // Run the GC and CC a few times to make sure that as much
             // as possible is freed.
             let numCycles = 3;
-            for (i = 0; i < numCycles; i++) {
+            for (let i = 0; i < numCycles; i++) {
               Cu.forceGC();
               Cu.forceCC();
             }
             aCallback();
           });
         };