999416 - Nodes can be in a native anonymous subtree without a root

Status: NEW

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

Description

[Aryeh Gregor (:ayg) (no longer with Mozilla)]

(In reply to :Aryeh Gregor from bug 805668 comment #35)
> (In reply to :Ehsan Akhgari (lagging on bugmail, needinfo? me!) from bug 805668 comment #31)
> >   if (IsInAnonymousSubtree()) {
> >   ...
> >     while (!root->IsRootOfNativeAnonymousSubtree()) {
> >       root = root->GetParent();
> >       // I don't think root should ever be null here, perhaps MOZ_ASSERT
> > that?
> 
> I did, and it can.  See try run -- a test crashes because of this assert. 
> In a debugger, I find that "this" is a text node with contents
> 
> foo\245\245\245\245\245--> \245\245\245\245\n
> <--\245\245\245MochiKitStatus\245\245Passed: Failed: Todo:
> \245\245\n\n\nZZZZZZZZZZZZZfirefox
> 
> with mFlags 546 (evidently NODE_IS_IN_NATIVE_ANONYMOUS_SUBTREE |
> NODE_IS_EDITABLE), and its parent is a DocumentFragment with mFlags 2.  This
> is all gibberish to me.  What's the right thing to do, break from the loop
> if root is null?  File a bug?

(In reply to :Ehsan Akhgari (lagging on bugmail, needinfo? me!) from bug 805668 comment #36)
> I bet this is coming from:
> http://mxr.mozilla.org/mozilla-central/source/dom/events/test/test_bug328885.
> html?force=1#88
> 
> It seems like in that test, the range object selects the native anonymous
> text node inside the input element and extracts it into a document fragment,
> which is what causes the GetParent() assertion to fail.  I _think_ the right
> thing to do is to make sure selectNodeContents() doesn't delve into the
> native anon subtree of input elements.  Please file a follow-up on that and
> get rid of the assertion for now (preferably with a comment pointing to the
> follow-up bug.)  Thanks.

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.