RESOLVED FIXED 65887
handle child margin values when flexing 
https://bugs.webkit.org/show_bug.cgi?id=65887
Summary handle child margin values when flexing
Tony Chang
Reported 2011-08-08 16:13:25 PDT
handle child margin values when flexing
Attachments
Patch (10.21 KB, patch)
2011-08-08 16:15 PDT, Tony Chang 		no flags
DetailsFormatted DiffDiff
Patch (10.27 KB, patch)
2011-08-08 16:22 PDT, Tony Chang 		no flags
DetailsFormatted DiffDiff
Patch (10.86 KB, patch)
2011-08-22 11:18 PDT, Tony Chang 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Tony Chang
Comment 1 2011-08-08 16:15:18 PDT
Created attachment 103309 [details] Patch
Gyuyoung Kim
Comment 2 2011-08-08 16:18:34 PDT
Comment on attachment 103309 [details] Patch Attachment 103309 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/9335146
WebKit Review Bot
Comment 3 2011-08-08 16:19:58 PDT
Comment on attachment 103309 [details] Patch Attachment 103309 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/9330634
Early Warning System Bot
Comment 4 2011-08-08 16:22:09 PDT
Comment on attachment 103309 [details] Patch Attachment 103309 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/9332197
Tony Chang
Comment 5 2011-08-08 16:22:44 PDT
Created attachment 103310 [details] Patch
Dave Hyatt
Comment 6 2011-08-22 10:39:13 PDT
Comment on attachment 103310 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=103310&action=review > Source/WebCore/rendering/RenderBox.cpp:1637 > +#if ENABLE(CSS3_FLEXBOX) > + && !cb->isFlexibleBox() > +#endif > + && !cb->isDeprecatedFlexibleBox()) I imagine this will come up again, so it seems like a method that wrapped both of these would be useful. Then you don't have ifdefs at all the call sites either.
Tony Chang
Comment 7 2011-08-22 11:18:02 PDT
Created attachment 104700 [details] Patch
Dave Hyatt
Comment 8 2011-08-22 13:33:09 PDT
Comment on attachment 104700 [details] Patch r=me
Tony Chang
Comment 9 2011-08-22 14:49:16 PDT
Committed r93547: <http://trac.webkit.org/changeset/93547>
Note You need to log in before you can comment on or make changes to this bug.