Closed Bug 772383 Opened 12 years ago Closed 12 years ago

Intermittent leak in browser_aboutCrashes.js

Categories

(Toolkit :: Crash Reporting, defect)

Product:
Toolkit
Component:
Crash Reporting
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
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: 12 years ago
Resolution: --- → WORKSFORME
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.