Created attachment 507580
Framedump (for the asserion in comment 99)
The last assertion comes from removing the child of a first-letter frame
(0x7fffde659138 blue), on the next reflow we call DrainOverflowFrames
which asserts that the principal list is non-empty when there is an
non-empty overflow list.
I think this is just my fault of not implementing Boris' suggested fix
quite right...
+ next->GetParent() == f->GetParent()) {
should have been
+ GetParent() == f->GetParent()) {
which would avoid doing the RemoveFrame optimization in this case.
Created attachment 507580
Framedump (for the asserion in comment 99)
The last assertion comes from removing the child of a first-letter frame
(0x7fffde659138 blue), on the next reflow we call DrainOverflowFrames
which asserts that the principal list is non-empty when there is an
non-empty overflow list.
I think this is just my fault of not implementing Boris' suggested fix
quite right...
+ next->GetParent() == f->GetParent()) {
should have been
+ GetParent() == f->GetParent()) {
which would avoid doing the RemoveFrame optimization in this case.