[webkit-reviews] review granted: [Bug 68129] change RenderFlexibleBox to act on logical coordinates : [Attachment 108063] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 20 15:33:25 PDT 2011

Dave Hyatt <hyatt at apple.com> has granted Ojan Vafai <ojan at chromium.org>'s
request for review:
Bug 68129: change RenderFlexibleBox to act on logical coordinates

Attachment 108063: Patch

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=108063&action=review


> Source/WebCore/rendering/RenderBlock.h:214
> +    void setLogicalLocationForChild(RenderBox* child, const LayoutPoint&);
>      void setLogicalLeftForChild(RenderBox* child, LayoutUnit logicalLeft,
ApplyLayoutDeltaMode = DoNotApplyLayoutDelta);
>      void setLogicalTopForChild(RenderBox* child, LayoutUnit logicalTop,
ApplyLayoutDeltaMode = DoNotApplyLayoutDelta);

Ultimately you'll probably want to support layout deltas for flexible box
children. You should file a followup bug about this.

> Source/WebCore/rendering/RenderFlexibleBox.cpp:324
> +	   setLogicalLocationForChild(child, IntPoint(logicalLeft,

You should probably have a placeChild method like the old flexible box did that
handles repainting. Right now you're moving without repainting, which will
result in repaint issues if the child moves without having to get a layout

More information about the webkit-reviews mailing list