[webkit-reviews] review requested: [Bug 47512] Add support for decoding WebP image : [Attachment 70489] add decoding support for webp image

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 11 16:04:57 PDT 2010


Pascal Massimino <pascal.massimino at gmail.com> has asked  for review:
Bug 47512: Add support for decoding WebP image
https://bugs.webkit.org/show_bug.cgi?id=47512

Attachment 70489: add decoding support for webp image
https://bugs.webkit.org/attachment.cgi?id=70489&action=review

------- Additional Comments from Pascal Massimino <pascal.massimino at gmail.com>
Hi,

this patch adds a sub-class of ImageDecoder that can handle
decoding of WebP images, which consists of VP8-compressed
data inside a RIFF container. It uses the light-weight decoding
library available from http://www.webmproject.org/code/.
Support should be disabled all across the board for now
(thanks to ENABLE_WEBP=0) unless i overlooked something.

WebP images are partially recognized thanks to the presence
of "RIFF" signature in the first 4-bytes. This is far from
perfect but should rule out most of spurious instantiations
of the decoding object.

I tried to change every configuration files i thought were
relevant by getting inspiration from the jpeg case. Hope
i didn't miss some!

thanks for the review,
Pascal

[ more info at http://code.google.com/speed/webp ]


More information about the webkit-reviews mailing list