Closed Bug 772383 Opened 13 years ago Closed 13 years ago

Intermittent leak in browser_aboutCrashes.js

Categories

(Toolkit :: Crash Reporting, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: emorley, Unassigned)

References

Details

(Keywords: intermittent-failure, memory-leak)

(Broken out from bug 754804) TEST-UNEXPECTED-FAIL | ShutdownLeaks | leaked 7 DOMWindow(s) and 1 DocShell(s) until shutdown [toolkit/crashreporter/test/browser/browser_aboutCrashes.js] 1 window(s) [url = about:blank]
Do you have a link to a log that shows this failing? The test creates a tab, loads about:crashes in that tab, then adds an onload listener which calls executeSoon to verify the tab's contents and then close the tab before calling finish(): http://mxr.mozilla.org/mozilla-central/source/toolkit/crashreporter/test/browser/browser_aboutCrashes.js#9 Not sure if there's something missing there, or if it could be made more reliable.
Whoops, sorry missed out the log. Now to find it again in bug 754804 :-s
TEST-START | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | directory service provider registered ok ++DOCSHELL 14A9A418 == 30 [id = 3443] ++DOMWINDOW == 110 (115E65C8) [serial = 9160] [outer = 00000000] ++DOMWINDOW == 111 (12DF8788) [serial = 9161] [outer = 115E6578] WARNING: NS_ENSURE_TRUE(mMutable) failed: file e:/builds/moz2_slave/m-in-w32-dbg/build/netwerk/base/src/nsSimpleURI.cpp, line 258 WARNING: NS_ENSURE_TRUE(mMutable) failed: file e:/builds/moz2_slave/m-in-w32-dbg/build/netwerk/base/src/nsSimpleURI.cpp, line 258 ++DOMWINDOW == 112 (11A1AF40) [serial = 9162] [outer = 0AB1BF78] ^^^ This is the window that's leaking. ++DOMWINDOW == 113 (13E26B18) [serial = 9163] [outer = 115E6578] TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | about:crashes loaded TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | about:crashes lists correct number of crash reports TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | 0: crash ID is correct TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | 1: crash ID is correct TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | 2: crash ID is correct TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | 3: crash ID is correct TEST-PASS | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | 4: crash ID is correct WARNING: NS_ENSURE_TRUE(mMutable) failed: file e:/builds/moz2_slave/m-in-w32-dbg/build/netwerk/base/src/nsSimpleURI.cpp, line 258 INFO TEST-END | chrome://mochitests/content/browser/toolkit/crashreporter/test/browser/browser_aboutCrashes.js | finished in 830ms I don't know how to diagnose that further.
Hasn't shown up on trunk after bug 728294 landed, so going to call this a false positive and mark WFM for now.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.