[webkit-reviews] review granted: [Bug 83426] Don't expose internal CSSValues in API : [Attachment 136137] try to fix build

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Apr 7 14:13:10 PDT 2012


Andreas Kling <kling at webkit.org> has granted Antti Koivisto <koivisto at iki.fi>'s
request for review:
Bug 83426: Don't expose internal CSSValues in API
https://bugs.webkit.org/show_bug.cgi?id=83426

Attachment 136137: try to fix build
https://bugs.webkit.org/attachment.cgi?id=136137&action=review

------- Additional Comments from Andreas Kling <kling at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=136137&action=review


Very cool. r=me.
(Though you'll need to do something about the win failure.)

> Source/WebCore/css/CSSImageSetValue.cpp:146
> +    // Non-CSSValueList data is not accesible through CSS OM, no need to
clone.

Typo accessible.

> Source/WebCore/css/CSSValue.h:38
> +// Please don't expose more CSSValue types to the web.

Hear hear. Perhaps we could even get rid of WebKitCSSTransformValue.idl and
WebKitCSSFilterValue.idl.

> Source/WebCore/css/RGBColor.cpp:42
> +    // Unique instances are safe.

Don't think this comment is needed.


More information about the webkit-reviews mailing list