Last Comment Bug 664467 - Unlink content/html some more
: Unlink content/html some more
Product: Core
Classification: Components
Component: DOM: Core & HTML (show other bugs)
: Trunk
: All All
: -- normal with 1 vote (vote)
: ---
Assigned To: Olli Pettay [:smaug] (reviewing overload)
Depends on:
Blocks: strongparent
  Show dependency treegraph
Reported: 2011-06-15 09:33 PDT by Olli Pettay [:smaug] (reviewing overload)
Modified: 2011-06-21 14:47 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (5.46 KB, patch)
2011-06-15 09:33 PDT, Olli Pettay [:smaug] (reviewing overload)
jonas: review+
Details | Diff | Splinter Review
+files and filelist (5.73 KB, patch)
2011-06-21 14:11 PDT, Olli Pettay [:smaug] (reviewing overload)
no flags Details | Diff | Splinter Review

Description Olli Pettay [:smaug] (reviewing overload) 2011-06-15 09:33:40 PDT
Created attachment 539563 [details] [diff] [review]

I need to re-verify if there are still other cases when we should
unlink, but I'll file followups when needed.
Comment 1 Jonas Sicking (:sicking) No longer reading bugmail consistently 2011-06-15 14:17:48 PDT
Comment on attachment 539563 [details] [diff] [review]

Review of attachment 539563 [details] [diff] [review]:

r=me either way.

::: content/html/content/src/nsHTMLInputElement.cpp
@@ +687,5 @@
> +                                                  nsGenericHTMLFormElement)
> +  //XXX should unlink more?

Wouldn't hurt to do mFiles and mFileList as well. It'd be needed if we make files trace their wrapper when an expando is set.

Also, who owns and traverses mInputData.mState?
Comment 2 Olli Pettay [:smaug] (reviewing overload) 2011-06-21 13:40:22 PDT
mInputData.mState is a reference variable  I don't see why it should be
Comment 3 Olli Pettay [:smaug] (reviewing overload) 2011-06-21 13:41:40 PDT
Er, I was looking at wrong code.
Comment 4 Olli Pettay [:smaug] (reviewing overload) 2011-06-21 14:11:15 PDT
Created attachment 540883 [details] [diff] [review]
+files and filelist
Comment 5 Olli Pettay [:smaug] (reviewing overload) 2011-06-21 14:47:02 PDT

the input element traverses mInputData.mState.

Note You need to log in before you can comment on or make changes to this bug.