[webkit-reviews] review denied: [Bug 23147] Introduce Skia to WebKit : [Attachment 26470] Add WebCore/platform/graphics/skia files

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied Dimitri Glazkov (Google)
<dglazkov at chromium.org>'s request for review:
Bug 23147: Introduce Skia to WebKit
https://bugs.webkit.org/show_bug.cgi?id=23147

Attachment 26470: Add WebCore/platform/graphics/skia files
https://bugs.webkit.org/attachment.cgi?id=26470&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
AffineTransformSkia.cpp needs to be renamed to TransformationMatrixSkia.cpp
(and the class too) to match recent renames upstream.

I don't understand what GdkSkia.cpp is for.  Seems it should be in its own
separate patch.

WebKit style would use coundUsed instead of count_used

We don't use c-style casts in C++ code:
+    SkColor* colors = (SkColor*)storage.get();

GraphicsContextSkia needs its own patch.  It's too big to grok as part of this
one.

Image buffer probably too.

== stopped reading ==