Summary: | Implement absolutely positioned flex items | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> | ||||||
Component: | Layout and Rendering | Assignee: | Tony Chang <tony> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | eric, ojan, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 62048 | ||||||||
Attachments: |
|
Description
Tony Chang
2012-08-13 00:00:29 PDT
Created attachment 164954 [details]
Patch
Comment on attachment 164954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=164954&action=review > Source/WebCore/rendering/RenderFlexibleBox.cpp:962 > +size_t RenderFlexibleBox::numberOfInFlowPositionedChildren(const OrderedFlexItemList& children) const Nit: could make this a static and not need to add it to the header. Comment on attachment 164954 [details] Patch Attachment 164954 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13958259 Created attachment 164958 [details]
Patch for landing
Comment on attachment 164954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=164954&action=review >> Source/WebCore/rendering/RenderFlexibleBox.cpp:962 >> +size_t RenderFlexibleBox::numberOfInFlowPositionedChildren(const OrderedFlexItemList& children) const > > Nit: could make this a static and not need to add it to the header. I considered that, but that involves making OrderedFlexItemList non-private, which would still touch the header and seems worse (only RenderFlexbox should be using this type). Comment on attachment 164954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=164954&action=review >>> Source/WebCore/rendering/RenderFlexibleBox.cpp:962 >>> +size_t RenderFlexibleBox::numberOfInFlowPositionedChildren(const OrderedFlexItemList& children) const >> >> Nit: could make this a static and not need to add it to the header. > > I considered that, but that involves making OrderedFlexItemList non-private, which would still touch the header and seems worse (only RenderFlexbox should be using this type). Ah, right. NM. Comment on attachment 164958 [details] Patch for landing Clearing flags on attachment: 164958 Committed r129154: <http://trac.webkit.org/changeset/129154> All reviewed patches have been landed. Closing bug. |