399712 - [FIX]"ASSERTION: Native anonymous node with wrong binding parent"

Status: RESOLVED FIXED

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

Description

[Jesse Ruderman]

Attachment: testcase

When I load the testcase and wait a few seconds, I see:

###!!! ASSERTION: Native anonymous node with wrong binding parent: '!aContent->IsNativeAnonymous()', file /Users/jruderman/trunk/mozilla/content/base/src/nsContentUtils.cpp, line 3702

I'm not sure this is a recent regression, but if it is, I suspect bug 378866.

Comment 1

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

That assertion was just added in Bug 345609.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Well, this is fun.  Spellcheck is getting computed style on a native anonymous node that's already been removed from the document (because the relevant frame is gone).  So the binding parent is null, while IsNativeAnonymous() is true.

I filed bug 399791 on spell-check.  I guess we should make this a warning for now, until that's fixed, but in general, I do think we should be asserting in this situation.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: "fix" to quiet down the assert

Comment 4

[Johnny Stenback (:jst)]

Comment on attachment 284859 [details] [diff] [review]
"fix" to quiet down the assert

Fun. r+sr+a=jst

Comment 5

[Boris Zbarsky [:bzbarsky]]

Fixed.

Comment 6

[Jesse Ruderman]

Crashtest checked in.  (I have the testcase run for three seconds.)