Closed Bug 8159 Opened 25 years ago Closed 25 years ago

[MLK] XUL elements leak when events are processed

Categories

(Core :: XUL, defect, P3)

Product:
Core
Component:
XUL
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: waterson, Assigned: waterson)

References

Details

(Whiteboard: Back to bruce, waterson for verification...) 

We currently leak some or all of the XUL nodes that get involved in processing
a DOM event; i.e., have their HandleDOMEvent() method called on them. I have no
idea why; creating this bug to track it.
Status: NEW → ASSIGNED
Summary: XUL elements leak when events are processed → [MLK] XUL elements leak when events are processed
Target Milestone: M8
Target Milestone: M8 → M9
Slipping to M9.
Bruce, I just took a look at your 19990726.log. It is my belief that we are
leaking RDF datasources because we leak each element in the content model that
has an event listener attached to it.

??? -> RDFElementImpl -> CompositeDataSource -> RDFXMLDataSource, etc.

Anyway, if anyone could help me dig into this I'd be much obliged.
Blocks: 8836
Whiteboard: working on this.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Turned out to be circularity between content model builder and composite
datasource, introduced when I changed the "nsIRDFObserver" ownership model. It
now appears (knock on wood) that we are actually starting to release the
content models! Woohoo!
This is such good news that I will endure some extra hours in the office this
weekend to try and verify this.
bruce & waterson, could you please verify this?  Thanks!
-Chris
Whiteboard: working on this. → Back to bruce, waterson for verification...
QA Contact: ckritzer → waterson
Assigning to originator per new organizational structure.
QA Contact: waterson → ckritzer
Changing QA Contact back to ckritzer@netscape.com
BULK MOVE: Changing component from XUL to XP Toolkit/Widgets: XUL.  XUL 
component will be deleted.
Component: XUL → XP Toolkit/Widgets: XUL
Massive QA Contact update.
QA Contact: ckritzer → jrgm
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.