Leaking nsXULDocuments on Mac

RESOLVED FIXED

Status

P3
normal
RESOLVED FIXED
19 years ago
14 years ago

People

(Reporter: sfraser_bugs, Assigned: sfraser_bugs)

Tracking

Trunk
PowerPC
Mac System 8.5

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

19 years ago
On Mac (and maybe elsewhere) we are leaking nsXULDocuments. With a breakpoint in 
~nsXULDocument, I see only the docs for sidebar panels being freed, but none for 
any of the top-level windows.
(Assignee)

Comment 1

19 years ago
My suspision is that nsEventStateManager is leaking nsXULDocuments. It seems to 
be holding owning refs to them via mDocument, and also in gLastFocusedDocument.
(Assignee)

Comment 2

19 years ago
nsEventStateMangager did not implement nsISupportsWeakReference, so was being 
held onto as observers of XPCOM shutdown. Fixing that fixes some ESM leakage. But 
I'm still leaking nsXULDocuments.

nsXULDocument::GetScriptObject() and nsXBLBinding=::GetScriptObject() are now
looking suspicious. The JS object that they return seems to be holding a ref to 
the nsXULDocument, yet I don't see where this ref goes away and neither do any 
kind of free or release on their mScriptObject. Does this ref go away when JS 
contexts are destroyed?
(Assignee)

Comment 3

19 years ago
This was fixed by the fix for 38869.
Assignee: waterson → sfraser
(Assignee)

Comment 4

19 years ago
Fixed!
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Comment 5

19 years ago
Sorry for the spam.  New QA Contact for Browser General.  Thanks for your help
Joseph (good luck with the new job) and welcome aboard Doron Rosenberg
QA Contact: jelwell → doronr

Comment 6

19 years ago
Simon, can you suggest/update the QA contact for this bug.  Thanks
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.