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

Issue 1044433002: Add SkWebpCodec, for decoding .webp images. (Closed)

Created:
5 years, 8 months ago by scroggo
Modified:
5 years, 5 months ago
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Add SkWebpCodec, for decoding .webp images. Based on SkImageDecoder_libwebp. TODO: Support YUV? (Longer term - may influence our API for SkImageGenerator) BUG=skia:3257 Committed: https://skia.googlesource.com/skia/+/6f5e619b877b0dc1a42910637a6ffe37add98001

Patch Set 1 #

Patch Set 2 : peek, handle rewind, add tests #

Patch Set 3 : Remove large stack allocation. #

Patch Set 4 : Call newer version of peek. #

Patch Set 5 : Support BGRA and ARGB for real. #

Patch Set 6 : rebase #

Total comments: 22

Patch Set 7 : Respond to jzern's comments. #

Patch Set 8 : Add a FIXME regarding incomplete input. #

Total comments: 10

Patch Set 9 : Allow DM to test decoding webp files. #

Patch Set 10 : Rebase #

Patch Set 11 : Fix conversion from float to int. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+249 lines, -2 lines) Patch
M dm/DM.cpp View 1 2 3 4 5 6 7 8 9 1 chunk +2 lines, -2 lines 0 comments Download
M gyp/codec.gyp View 1 2 3 4 5 6 7 8 9 10 2 chunks +2 lines, -0 lines 0 comments Download
M src/codec/SkCodec.cpp View 1 2 3 4 5 6 7 8 9 2 chunks +2 lines, -0 lines 0 comments Download
A src/codec/SkWebpCodec.h View 1 chunk +38 lines, -0 lines 0 comments Download
A src/codec/SkWebpCodec.cpp View 1 2 3 4 5 6 7 8 9 10 1 chunk +200 lines, -0 lines 0 comments Download
M tests/CodexTest.cpp View 1 2 3 4 5 6 7 8 9 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 36 (11 generated)
scroggo
5 years, 8 months ago (2015-03-30 18:53:18 UTC) #2
scroggo
jzern@, can you take a look at this and see if our use of webp ...
5 years, 8 months ago (2015-04-02 21:31:55 UTC) #4
jzern
On 2015/04/02 21:31:55, scroggo wrote: > jzern@, can you take a look at this and ...
5 years, 8 months ago (2015-04-02 22:56:27 UTC) #5
msarett
Just a few questions. https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp#newcode168 src/codec/SkWebpCodec.cpp:168: config.output.u.RGBA.rgba = (uint8_t*) dst; Does ...
5 years, 8 months ago (2015-04-03 12:28:13 UTC) #6
scroggo
On 2015/04/02 22:56:27, jzern wrote: > On 2015/04/02 21:31:55, scroggo wrote: > > jzern@, can ...
5 years, 8 months ago (2015-04-03 14:14:12 UTC) #7
jzern
On 2015/04/03 14:14:12, scroggo wrote: > On 2015/04/02 22:56:27, jzern wrote: > > On 2015/04/02 ...
5 years, 8 months ago (2015-04-03 19:15:30 UTC) #8
jzern
https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp#newcode15 src/codec/SkWebpCodec.cpp:15: // http://www.webmproject.org/code/#libwebp_webp_image_decoder_library #libwebp-webp-image-library https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp#newcode16 src/codec/SkWebpCodec.cpp:16: // http://review.webmproject.org/gitweb?p=libwebp.git long dead. ...
5 years, 8 months ago (2015-04-03 19:15:49 UTC) #9
jzern
https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/100001/src/codec/SkWebpCodec.cpp#newcode183 src/codec/SkWebpCodec.cpp:183: return kInvalidInput; // Or incomplete? On 2015/04/03 14:14:12, scroggo ...
5 years, 8 months ago (2015-04-03 19:21:34 UTC) #10
scroggo
On 2015/04/03 19:15:30, jzern wrote: > Sorry about that, I wasn't paying attention... NP, it ...
5 years, 8 months ago (2015-04-03 20:00:19 UTC) #11
jzern
lgtm
5 years, 8 months ago (2015-04-03 21:35:09 UTC) #12
scroggo
Derek and/or Matt, can you take a look? I think I need a Skia member ...
5 years, 8 months ago (2015-04-06 14:02:09 UTC) #14
djsollen
https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp#newcode67 src/codec/SkWebpCodec.cpp:67: // best type, when Skia currently cannot draw unpremul ...
5 years, 8 months ago (2015-04-06 14:36:28 UTC) #15
scroggo
https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp#newcode67 src/codec/SkWebpCodec.cpp:67: // best type, when Skia currently cannot draw unpremul ...
5 years, 8 months ago (2015-04-06 16:34:43 UTC) #16
jzern
https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp#newcode67 src/codec/SkWebpCodec.cpp:67: // best type, when Skia currently cannot draw unpremul ...
5 years, 8 months ago (2015-04-06 21:30:34 UTC) #17
scroggo
https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp#newcode107 src/codec/SkWebpCodec.cpp:107: dim.fHeight *= desiredScale; On 2015/04/06 21:30:34, jzern wrote: > ...
5 years, 8 months ago (2015-04-07 18:46:10 UTC) #18
jzern
https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp File src/codec/SkWebpCodec.cpp (right): https://codereview.chromium.org/1044433002/diff/130006/src/codec/SkWebpCodec.cpp#newcode107 src/codec/SkWebpCodec.cpp:107: dim.fHeight *= desiredScale; On 2015/04/07 18:46:10, scroggo wrote: > ...
5 years, 8 months ago (2015-04-07 20:04:23 UTC) #19
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1044433002/170001
5 years, 6 months ago (2015-06-18 17:25:32 UTC) #22
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED, http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_64-Debug-Trybot/builds/1612)
5 years, 6 months ago (2015-06-18 17:29:05 UTC) #24
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1044433002/190001
5 years, 6 months ago (2015-06-18 18:44:59 UTC) #27
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 6 months ago (2015-06-18 18:50:36 UTC) #29
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1044433002/190001
5 years, 6 months ago (2015-06-18 19:22:59 UTC) #31
commit-bot: I haz the power
Note for Reviewers: The CQ is waiting for an approval. If you believe that the ...
5 years, 6 months ago (2015-06-18 19:23:00 UTC) #32
scroggo
Can I get a Skia lgtm? With this, and some more code to do subsets ...
5 years, 6 months ago (2015-06-18 19:31:48 UTC) #34
msarett
lgtm
5 years, 6 months ago (2015-06-18 19:52:30 UTC) #35
commit-bot: I haz the power
5 years, 6 months ago (2015-06-18 19:53:47 UTC) #36
Message was sent while issue was closed.
Committed patchset #11 (id:190001) as
https://skia.googlesource.com/skia/+/6f5e619b877b0dc1a42910637a6ffe37add98001

Powered by Google App Engine
This is Rietveld 408576698