Jesse Ruderman Reporter
	Description • 18 years ago

Loading the testcase triggers:

###!!! ASSERTION: Should have dealt with kids that have NS_FRAME_IS_SPECIAL elsewhere: 'aChildPseudo != nsCSSAnonBoxes::mozAnonymousBlock && aChildPseudo != nsCSSAnonBoxes::mozAnonymousPositionedBlock', nsFrame.cpp, line 5494

This assertion was added in bug 323656.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 1 • 18 years ago

Mm.... so MathML is reusing the mozAnonymousBlock pseudo that really only {ib} splits should be using.  I suppose we can comment out the assertion until after MathML switches to inline-table, or give MathML its own pseudo.

I'd prefer the latter approach, frankly, but on branches maybe we need to do the former.

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 2 • 18 years ago

David, let me know whether you have a preference, ok?

	David Baron :dbaron:
	Comment 3 • 18 years ago

Seems like its own pseudo is better.

I don't think content can style these pseudos, so I don't think there should be any issues for branch.

	David Baron :dbaron:
	Comment 5 • 17 years ago

Comment on attachment 261912 [details] [diff] [review]
Let's do that.

r+sr=dbaron

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 6 • 17 years ago

Fixed on trunk.

	Daniel Veditz [:dveditz]
	Comment 7 • 17 years ago

Comment on attachment 261912 [details] [diff] [review]
Let's do that.

approved for 1.8.0.12 and 1.8.1.4, a=dveditz for release-drivers

	Boris Zbarsky [:bzbarsky] Assignee
	Comment 8 • 17 years ago

Checked in on the branches.

	Jesse Ruderman Reporter
	Comment 9 • 17 years ago

Crashtest checked in.