326635 - node.setUserData("foo", anyNode, null) leaks GC roots and DOMWindows

Status: RESOLVED DUPLICATE of bug 324871

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

Description

[Jesse Ruderman]

JS engine warning: leaking GC root 'XPCVariant::mJSVal' at 0x2459f294
JS engine warning: 1 GC root remains after destroying the JSRuntime.
                   This root may point to freed memory. Objects reachable
                   through it have not been finalized.

Comment 1

[Jesse Ruderman]

Attachment: testcase

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Duplicate of bug 324871?

Comment 3

[Jesse Ruderman]

*** This bug has been marked as a duplicate of 324871 ***

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

So I actually don't immediately see why the document is being kept alive in this case, although I suspect from the warning that fixing this requires treating XPCVariant like bug 241518 treated nsXPCWrappedJS -- by marking rather than rooting.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

This appears to be a bug that was created since the 1.8 branch, since bug 326635 landed after the branch; therefore maybe it can wait for graydon's cycle collector to fix the problem.