[webkit-reviews] review granted: [Bug 78772] flex-wrap:nowrap should be flex-wrap:none : [Attachment 127470] ProposedPatch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 16 16:56:34 PST 2012


Ojan Vafai <ojan at chromium.org> has granted Joe Thomas
<joethomas at motorola.com>'s request for review:
Bug 78772: flex-wrap:nowrap should be flex-wrap:none
https://bugs.webkit.org/show_bug.cgi?id=78772

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

------- Additional Comments from Ojan Vafai <ojan at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=127470&action=review


Thanks for the fix!

> Source/WebCore/ChangeLog:10
> +	   Modified the existing test case.

FYI: In the future you don't need to include this description. You can just
delete the no new tests line. In cases where there actually isn't test coverage
*and* you're not adding tests, then you do need an explanation of why tests
aren't needed.


More information about the webkit-reviews mailing list