[webkit-reviews] review granted: [Bug 69460] Turn ENABLE_VIDEO_TRACK on for chromium. : [Attachment 109875] fix for compiler () preferences

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 5 17:08:18 PDT 2011


David Levin <levin at chromium.org> has granted Anna Cavender
<annacc at chromium.org>'s request for review:
Bug 69460: Turn ENABLE_VIDEO_TRACK on for chromium.
https://bugs.webkit.org/show_bug.cgi?id=69460

Attachment 109875: fix for compiler () preferences
https://bugs.webkit.org/attachment.cgi?id=109875&action=review

------- Additional Comments from David Levin <levin at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=109875&action=review


> Source/WebCore/html/track/WebVTTParser.cpp:343
> +	       || (m_token.name().size() == 1 && m_token.name()[0] == 'v'))

Since the "then" has multiple lines, it should have braces around it.


More information about the webkit-reviews mailing list