Open Bug 903678 Opened 12 years ago Updated 3 years ago

Consider removing ReleaseWrapper calls from DestroyContent

Categories

(Core :: DOM: Core & HTML, defect, P5)

defect

Tracking

()

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
Summary: Consider removing ReleaseWrapper calls from ContentDestroy → Consider removing ReleaseWrapper calls from DestroyContent
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
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.