Closed Bug 1308998 Opened 8 years ago Closed 8 years ago

Fix leaked i variable on window from browser-test.js

Categories

(Testing :: Mochitest, defect)

Product:
Testing
Component:
Mochitest
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox50 fixed, firefox51 fixed, firefox52 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla52
Tracking Flags:
Tracking Status
firefox50 --- fixed
firefox51 --- fixed
firefox52 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

Details

Attachments

(1 file)

Bug 1308998 - remove leaked i variable in browser-test.js;
58 bytes, text/x-review-board-request
mccr8
: review+
Details 
When running a single test several times (using --run-until-failure for instance), the second iteration fails with 

> test left unexpected property on window: i

This seems to be a regression from Bug 1305836
Comment on attachment 8799478 [details]
Bug 1308998 - remove leaked i variable in browser-test.js;

https://reviewboard.mozilla.org/r/84644/#review83248

Thanks.
Attachment #8799478 - Flags: review?(continuation) → review+
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/43d5e51ac48c
remove leaked i variable in browser-test.js;r=mccr8
https://hg.mozilla.org/mozilla-central/rev/43d5e51ac48c
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
status-firefox50: --- → affected
status-firefox51: --- → affected
Whiteboard: [checkin-needed-aurora][checkin-needed-beta]
Whiteboard: [checkin-needed-aurora][checkin-needed-beta] → [checkin-needed-beta]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: