Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(207)

Issue 10860057: Merge 125869 - Source/WebCore: libwebp-0.2.0: handle alpha channel if present (Closed)

Created:
8 years, 4 months ago by Noel Gordon
Modified:
8 years, 4 months ago
Reviewers:
commit-queue
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1229/
Visibility:
Public.

Description

Merge 125869 - Source/WebCore: libwebp-0.2.0: handle alpha channel if present https://bugs.webkit.org/show_bug.cgi?id=93430 Patch by Pascal Massimino <pascal.massimino@gmail.com>; on 2012-08-17 Reviewed by Adam Barth. updated the Layout test webp-image-decoding.html * platform/image-decoders/webp/WEBPImageDecoder.cpp: (outputMode): switch to premultiplied-RGB if needed (WebCore::WEBPImageDecoder::WEBPImageDecoder): (WebCore::WEBPImageDecoder::decode): use WebPGetFeatures() instead of WebPGetInfo(), to detect presence of alpha channel * platform/image-decoders/webp/WEBPImageDecoder.h: (WEBPImageDecoder): add a m_hasAlpha field LayoutTests: update layout test for new libwebp-0.2.0 https://bugs.webkit.org/show_bug.cgi?id=93430 Patch by Pascal Massimino <pascal.massimino@gmail.com>; on 2012-08-17 Reviewed by Adam Barth. * fast/images/resources/test2.webp: Added. lossy+alpha example * fast/images/resources/test3.webp: Added. lossless with alpha example * fast/images/webp-image-decoding-expected.txt: * fast/images/webp-image-decoding.html: Updated test. Will now decode test2.webp and test3.webp too. TBR=commit-queue@webkit.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126112

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -6 lines) Patch
A + LayoutTests/fast/images/resources/test2.webp View 0 chunks +-1 lines, --1 lines 0 comments Download
A + LayoutTests/fast/images/resources/test3.webp View 0 chunks +-1 lines, --1 lines 0 comments Download
M LayoutTests/fast/images/webp-image-decoding.html View 1 chunk +7 lines, -1 line 0 comments Download
M LayoutTests/fast/images/webp-image-decoding-expected.txt View 1 chunk +2 lines, -2 lines 0 comments Download
M Source/WebCore/platform/image-decoders/webp/WEBPImageDecoder.h View 1 chunk +1 line, -0 lines 0 comments Download
M Source/WebCore/platform/image-decoders/webp/WEBPImageDecoder.cpp View 5 chunks +23 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Noel Gordon
8 years, 4 months ago (2012-08-21 01:00:02 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698