Closed
Bug 398017
Opened 17 years ago
Closed 17 years ago
nsGenericElement and nsJSEventListener unlink improvements
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
People
(Reporter: sayrer, Assigned: sayrer)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
While working on a smaller version of bug 397206, I noticed this message in the DEBUG_CC messages:
nsCycleCollector: nsGenericElement 0x345eb430 in component 152
was not collected due to missing call to suspect, failure to unlink,
or deficiency in traverse that causes cycles referenced only from other
cycles to require multiple rounds of cycle collection
nsCycleCollector: nsGenericElement 0x345e9650 in component 95
was not collected due to missing call to suspect, failure to unlink,
or deficiency in traverse that causes cycles referenced only from other
cycles to require multiple rounds of cycle collection
nsCycleCollector: nsEventListenerManager 0x345eb460 in component 152
was not collected due to missing call to suspect, failure to unlink,
or deficiency in traverse that causes cycles referenced only from other
cycles to require multiple rounds of cycle collection
nsCycleCollector: nsEventListenerManager 0x345e9680 in component 95
was not collected due to missing call to suspect, failure to unlink,
or deficiency in traverse that causes cycles referenced only from other
cycles to require multiple rounds of cycle collection
nsCycleCollector: nsJSEventListener 0x345eb490 in component 152
was not collected due to missing call to suspect, failure to unlink,
or deficiency in traverse that causes cycles referenced only from other
cycles to require multiple rounds of cycle collection
nsCycleCollector: nsJSEventListener 0x345e96b0 in component 95
was not collected due to missing call to suspect, failure to unlink,
or deficiency in traverse that causes cycles referenced only from other
cycles to require multiple rounds of cycle collection
Assignee | ||
Comment 1•17 years ago
|
||
Attachment #282847 -
Flags: superreview+
Attachment #282847 -
Flags: review?(peterv)
Attachment #282847 -
Flags: review+
Assignee | ||
Updated•17 years ago
|
Attachment #282847 -
Flags: approval1.9?
Comment 2•17 years ago
|
||
Comment on attachment 282847 [details] [diff] [review]
unlink more things
The change in nsJSEventListener.cpp isn't needed anymore on trunk. Probably still want the change in nsGenericElement.cpp.
Comment 3•17 years ago
|
||
Comment on attachment 282847 [details] [diff] [review]
unlink more things
a=release drivers
Attachment #282847 -
Flags: approval1.9? → approval1.9+
Updated•17 years ago
|
Whiteboard: [needs landing]
Updated•17 years ago
|
Version: unspecified → Trunk
Updated•17 years ago
|
Whiteboard: [needs landing]
Assignee | ||
Comment 4•17 years ago
|
||
Attachment #282847 -
Attachment is obsolete: true
Assignee | ||
Updated•17 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•17 years ago
|
Flags: in-testsuite-
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•