[webkit-reviews] review granted: [Bug 83152] [GTK] Separate image encoding from dataURL construction : [Attachment 135575] Patch

[bugzilla-daemon at webkit.org]

Martin Robinson <mrobinson at webkit.org> has granted noel gordon
<noel.gordon at gmail.com>'s request for review:
Bug 83152: [GTK] Separate image encoding from dataURL construction
https://bugs.webkit.org/show_bug.cgi?id=83152

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

------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=135575&action=review


> Source/WebCore/platform/graphics/gtk/ImageBufferGtk.cpp:70
> +    base64Encode(reinterpret_cast<const char*>(buffer.get()), bufferSize,
base64Data);

It seems unusual that we need this reinpterpret_cast here. Perhaps you could
try removing this before landing?