[webkit-reviews] review granted: [Bug 67243] add a test for border/padding/margin on the flexbox container : [Attachment 105714] Patch

[bugzilla-daemon at webkit.org]

Tony Chang <tony at chromium.org> has granted Ojan Vafai <ojan at chromium.org>'s
request for review:
Bug 67243: add a test for border/padding/margin on the flexbox container
https://bugs.webkit.org/show_bug.cgi?id=67243

Attachment 105714: Patch
https://bugs.webkit.org/attachment.cgi?id=105714&action=review

------- Additional Comments from Tony Chang <tony at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=105714&action=review


I like more tests.

> LayoutTests/css3/flexbox/001.html:146
> +    <div data-expected-width="700" style="display: -webkit-flexbox;
padding-left: 10px; padding-right: 20px; border-left: 30px solid; border-right:
40px solid; margin-left: 50px; margin-right:60px;" class="horizontal-box ">

Can we drop the display: property?