Closed
Bug 711907
Opened 13 years ago
Closed 12 years ago
Rewrite browser_tabview_bug597980.js to be more fresh and modern, and less leaky
Categories
(Firefox Graveyard :: Panorama, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: philor, Unassigned)
References
Details
Clear back in bug 653080 comment 4 we started pointing the finger at browser_tabview_bug597980.js, but then it took us another 700 instances (many of which may well have been other things) to actually consider whether that might be it. https://tbpl.mozilla.org/?tree=Try&rev=8109f47f7bff is my control push, just the tip of mozilla-central, which got four leaks out of twenty runs on Linux64, and one leak out of twenty runs on OS X64. https://tbpl.mozilla.org/?tree=Try&rev=f7ea7fc1e3a4 is my experiment, disabling browser_tabview_bug597980.js, which got zero leaks out of twenty one runs on Linux64, and zero leaks out of twenty one runs on OS X64. Dunno what it is about the test that leaks the world (though, since I'm suspicious of any use of setTimeout, ever, with good reason, I'm suspicious of the double use here), but please make it stop. In the meantime, I'm disabling it with ttaubert's okay.
Updated•13 years ago
|
Whiteboard: [MemShrink]
![]() |
||
Updated•13 years ago
|
Whiteboard: [MemShrink]
Comment 1•13 years ago
|
||
We removed the MemShrink tag on the assumption that this is a problem with the test, and not the test exposing a problem with the browser.
Comment 2•13 years ago
|
||
Not sure why exactly the test was leaking, but besides just rewriting we need to at least remove these arbitrary setTimeout() calls and make sure to use waitForFocus() before using EventUtils methods.
Comment 3•12 years ago
|
||
Let's not spend time on this.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•