Summary: | small refactor of RenderFlexibleBox | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> | ||||||
Component: | New Bugs | Assignee: | Tony Chang <tony> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ojan, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Tony Chang
2011-12-05 12:32:25 PST
Created attachment 117918 [details]
Patch
Easy review. Makes the variable names more like the names in RenderFlexibleBox::layoutColumnReverse. Comment on attachment 117918 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117918&action=review > Source/WebCore/rendering/RenderFlexibleBox.cpp:627 > + IntPoint childLocation(shouldFlipMainAxis ? totalMainExtent - mainAxisOffset - childMainExtent : mainAxisOffset, > + crossAxisOffset + flowAwareMarginBeforeForChild(child)); WebKit formatting rules forbid this kind of lining up since it’s not renaming-infriendly. Created attachment 117928 [details]
Patch for landing
Comment on attachment 117928 [details] Patch for landing Clearing flags on attachment: 117928 Committed r102090: <http://trac.webkit.org/changeset/102090> All reviewed patches have been landed. Closing bug. |