505231 - Make nsBidiPresUtils use framelists

Status: NEW

(Core :: Layout, defect, P4)

Description

[Boris Zbarsky [:bzbarsky]]

There are some temp frame lists in SplitInlineAncestors; the frame list usage should be pushed up in the algorithm.

Comment 1

[Boris Zbarsky [:bzbarsky]]

So there is no "up" to push to, but really solving this in a world where child lists have last child pointers needs a method on nsIFrame that takes a location between two frames in a child list (so an nsFrameList::SplittableIterator or whatever we decide to call it) and truncates the list before the location, while returning the remaining frames.

Alternately, it requires that GetChildList() return a non-const reference (which means finally dealing with blockframe child storage, as well as generally exposing the child list directly)...

I'm tempted to go with making GetChildList() return non-const, if no one thinks the other approach is more palatable.