Open
Bug 903678
Opened 12 years ago
Updated 3 years ago
Consider removing ReleaseWrapper calls from DestroyContent
Categories
(Core :: DOM: Core & HTML, defect, P5)
Core
DOM: Core & HTML
Tracking
()
NEW
People
(Reporter: mccr8, Unassigned)
Details
There are three of these, with the comment "XXX We really should let cycle collection do this, but that currently still leaks (see https://bugzilla.mozilla.org/show_bug.cgi?id=406684).". That bug wasn't very helpful, but it is from 2007, so hopefully the CC is better now.
I don't know of any reason having these ReleaseWrappers is bad, but they seem a little sketchy. But I'm also not really sure what the goal of DestroyContent is.
I did a try run with all three removed and it looked fine, so maybe these can go:
https://tbpl.mozilla.org/?tree=Try&rev=0de6a6910f3c
Reporter | ||
Updated•12 years ago
|
Summary: Consider removing ReleaseWrapper calls from ContentDestroy → Consider removing ReleaseWrapper calls from DestroyContent
Comment 1•7 years ago
|
||
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046
Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.
If you have questions, please contact :mdaly.
Priority: -- → P5
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•