[webkit-reviews] review granted: [Bug 87462] Use StringBuilder in WebKitCSSTransformValue::customCssText() to allow code reuse with CSS Variables. : [Attachment 143977] Patch

[bugzilla-daemon at webkit.org]

Dimitri Glazkov (Google) <dglazkov at chromium.org> has granted Luke Macpherson
<macpherson at chromium.org>'s request for review:
Bug 87462: Use StringBuilder in WebKitCSSTransformValue::customCssText() to
allow code reuse with CSS Variables.
https://bugs.webkit.org/show_bug.cgi?id=87462

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

------- Additional Comments from Dimitri Glazkov (Google)
<dglazkov at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=143977&action=review


> Source/WebCore/css/WebKitCSSTransformValue.cpp:70
> +    if (m_type != UnknownTransformOperation) {

Do we need to ASSERT the expected range of m_type somewhere around here?