903678 - Consider removing ReleaseWrapper calls from DestroyContent

Status: NEW

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

Description

[Andrew McCreight [:mccr8]]

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

Comment 1

[Firefox Bug Husbandry Bot]

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.