664464 - Unlink XBL some more

Status: RESOLVED FIXED

(Core :: XBL, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 1

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 539560 [details] [diff] [review]
patch

Why is mNextBinding needed? Leaving just those pointers should never create a cycle unless we have some really horrible bugs.

r=me if you explain why that's unlinked.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

mNextBinding may have mContent pointing to element which keeps
document alive.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Though, ofc mNextBinding should unlink its mContent. 
But still, I'd like to keep unlinking "complete" when possible.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

http://hg.mozilla.org/mozilla-central/rev/db6d8ef5b27c

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

But that gets unlinked, right?