Closed
Bug 1308998
Opened 10 years ago
Closed 10 years ago
Fix leaked i variable on window from browser-test.js
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(firefox50 fixed, firefox51 fixed, firefox52 fixed)
RESOLVED
FIXED
mozilla52
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
Attachments
(1 file)
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 hidden (mozreview-request) |
Comment 2•10 years ago
|
||
| mozreview-review | ||
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
Comment 4•10 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Updated•10 years ago
|
status-firefox50:
--- → affected
status-firefox51:
--- → affected
Whiteboard: [checkin-needed-aurora][checkin-needed-beta]
Comment 5•10 years ago
|
||
| bugherder uplift | ||
Updated•10 years ago
|
Whiteboard: [checkin-needed-aurora][checkin-needed-beta] → [checkin-needed-beta]
Comment 6•10 years ago
|
||
| bugherder uplift | ||
Whiteboard: [checkin-needed-beta]
You need to log in
before you can comment on or make changes to this bug.
Description
•