Created attachment 507697
Framedump (for the assertion in comment 93)
The cause of the assertion/crash in comment 93 is the RemoveFrame we added
causes a lot of first-letter frames to become empty. Then we reflow one
of those and it can't handle that. (See bug 578977.)
The GetNextSibling() check we added should prevent this.
Created attachment 507697
Framedump (for the assertion in comment 93)
The cause of the assertion/crash in comment 93 is the RemoveFrame we added
causes a lot of first-letter frames to become empty. Then we reflow one
of those and it can't handle that. (See bug 578977.)
The GetNextSibling() check we added should prevent this.
The assertion in comment 95 looks like a bogus assert. I think it's a typo ->IsEmpty( )": hg.mozilla. org/mozilla- central/ annotate/ e98b94aa64fa/ layout/ generic/ nsFirstLetterFr ame.cpp# l364 ames() already assert this: hg.mozilla. org/mozilla- central/ annotate/ e98b94aa64fa/ layout/ generic/ nsContainerFram e.h#l627
that was really meant to be "overflowFrames
http://
I propose we simply remove it since StealOverflowFr
http://