RESOLVED FIXED Bug 81843
refactor flexbox in preparation for flex-line-pack 
https://bugs.webkit.org/show_bug.cgi?id=81843
Summary refactor flexbox in preparation for flex-line-pack
Tony Chang
Reported 2012-03-21 16:36:58 PDT
refactor flexbox in preparation for flex-line-pack
Attachments
Patch (15.67 KB, patch)
2012-03-21 16:40 PDT, Tony Chang 		no flags
DetailsFormatted DiffDiff
Patch for landing (15.63 KB, patch)
2012-03-21 17:02 PDT, Tony Chang 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tony Chang
Comment 1 2012-03-21 16:40:48 PDT
Created attachment 133142 [details] Patch
Tony Chang
Comment 2 2012-03-21 16:41:24 PDT
Let me know if this is too big. I can split it into two patches.
Ojan Vafai
Comment 3 2012-03-21 16:47:55 PDT
Comment on attachment 133142 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=133142&action=review > Source/WebCore/rendering/RenderFlexibleBox.cpp:1009 > + if (flexAlignForChild(child) == AlignBaseline && minMarginAfterBaseline) Nit: I'd move the minMarginAfterBaseline check to the outer for-loop and early-continue: if (!minMarginAfterBaseline) continue;
Tony Chang
Comment 4 2012-03-21 17:02:34 PDT
Created attachment 133144 [details] Patch for landing
WebKit Review Bot
Comment 5 2012-03-21 18:29:22 PDT
Comment on attachment 133144 [details] Patch for landing Clearing flags on attachment: 133144 Committed r111635: <http://trac.webkit.org/changeset/111635>
WebKit Review Bot
Comment 6 2012-03-21 18:29:26 PDT
All reviewed patches have been landed. Closing bug.
mitz
Comment 7 2012-03-21 21:36:13 PDT
(In reply to comment #5) > (From update of attachment 133144 [details]) > Clearing flags on attachment: 133144 > > Committed r111635: <http://trac.webkit.org/changeset/111635> This caused bug 81870.
mitz
Comment 8 2012-03-21 21:59:18 PDT
Reverted r111635 in <http://trac.webkit.org/r111652>.
Tony Chang
Comment 9 2012-03-22 10:16:38 PDT
Committed r111720: <http://trac.webkit.org/changeset/111720>
Note You need to log in before you can comment on or make changes to this bug.