Closed
Bug 609718
Opened 14 years ago
Closed 12 years ago
Assertion in browser_webconsole_chrome.js
Categories
(DevTools :: Console, defect, P2)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: ddahl, Unassigned)
Details
(Whiteboard: [dupe-me] [console-close-me] )
Attachments
(1 file)
(deleted),
text/plain
|
Details |
I se an assertion in this test:
browser_webconsole_chrome.js
###!!! ASSERTION: Oops! You're asking for a weak reference to an object that doesn't support that.: 'factoryPtr', file nsWeakReference.cpp, line 109
All checks pass, however.
Reporter | ||
Comment 1•14 years ago
|
||
This may only be with the lazy console patches from bug 568629 and the integration patch in bug 587734
Reporter | ||
Comment 2•14 years ago
|
||
Comment on attachment 488309 [details]
Assertion bt
Perhaps we should CC someone who might know this code well? Of course this code is Copyright (C) 1999 code:)
Comment 3•14 years ago
|
||
Are you sure the assertion is caused by hudservice code and that test specifically? That stack looks like nsIObserverService::removeObserver() being called from a pagehide handler (possibly caused by a tab close?), and I don't see any code doing that in HUDService.jsm or in the test.
Reporter | ||
Comment 4•14 years ago
|
||
(In reply to comment #3)
> Are you sure the assertion is caused by hudservice code and that test
> specifically? That stack looks like nsIObserverService::removeObserver() being
> called from a pagehide handler (possibly caused by a tab close?), and I don't
> see any code doing that in HUDService.jsm or in the test.
I was unsure, and saw this several times during test runs. It may not be hudservice code. Thanks for looking into this.
Reporter | ||
Comment 5•14 years ago
|
||
Also, I only noticed this after the test checks were finished and mochitest was about to load the next test.
Reporter | ||
Updated•14 years ago
|
Whiteboard: [dupe-me] [console-close-me]
Reporter | ||
Comment 7•14 years ago
|
||
This may have to do with the observers shutting down on shutdown
Whiteboard: [dupe-me] [console-close-me] → [dupe-me] [console-close-me]
Updated•12 years ago
|
Component: Developer Tools → Developer Tools: Console
Comment 8•12 years ago
|
||
Is this still valid? I would be inclined to say no...
Comment 9•12 years ago
|
||
Unlikely.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•