Add support for decoding WebP images

Add support for decoding WebP images

Add a wrapper around libwebp to decompress WebP images and re-encode them
to uncompressed TIFF (if small enough) or JPEG/PNG (dependending on image
features).

BUG=436897
Committed: https://crrev.com/987f84d0d84d4e59b36ce8532ddcaf174f1346cd
Cr-Commit-Position: refs/heads/master@{#308069}

[sdefresne, 2014-12-01 17:16:25]

droger: Please take a look

eugenebut: OWNERS of //ios/web
fbarchard: OWNERS of new dependency on //third_party/libwebp

[sdefresne, 2014-12-01 17:17:54]

sdefresne@chromium.org changed reviewers:
+ stuartmorgan@chromium.org
- eugenebut@chromium.org

[sdefresne, 2014-12-01 17:18:19]

-eugenebut, +stuartmorgan: OWNERS of //ios/web

[stuartmorgan, 2014-12-01 18:04:03]

Mostly-rubber-stamp LGTM since it's direct upstreaming. One header nit though.

https://codereview.chromium.org/771723002/diff/1/ios/web/public/webp_decoder.h
File ios/web/public/webp_decoder.h (right):

https://codereview.chromium.org/771723002/diff/1/ios/web/public/webp_decoder....
ios/web/public/webp_decoder.h:69: bool DoSendData();
Could you correct the lack of comments here while you're upstreaming it?

[droger, 2014-12-01 18:15:14]

lgtm

[sdefresne, 2014-12-02 08:45:13]

TBR-ing the dependency on //third_party/libwebp according to the comment in
//third_party/OWNERS.

[sdefresne, 2014-12-02 08:45:20]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2014-12-02 08:45:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/771723002/20001

[commit-bot: I haz the power, 2014-12-02 08:49:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-02 08:49:04]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/88530)
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  android_arm64_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sdefresne, 2014-12-02 09:00:48]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2014-12-02 09:02:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/771723002/40001

[commit-bot: I haz the power, 2014-12-02 09:05:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-02 09:05:31]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/88536)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/93159)
  linux_chromium_asan_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sdefresne, 2014-12-02 10:26:09]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2014-12-02 10:27:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/771723002/60001

[commit-bot: I haz the power, 2014-12-02 10:35:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-12-02 10:35:13]

Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[sdefresne, 2014-12-10 10:01:11]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2014-12-10 10:01:42]

The CQ bit was unchecked by sdefresne@chromium.org

[sdefresne, 2014-12-10 10:02:40]

fbarchard: can you review the new dependency on third_party/libwebp?

[fbarchard, 2014-12-10 18:24:56]

lgtm
Consider GN support.

[sdefresne, 2014-12-10 18:33:30]

On 2014/12/10 at 18:24:56, fbarchard wrote:
> lgtm
> Consider GN support.

Thank you for the review. iOS build currently does not supports GN (there are
some missing features).

[sdefresne, 2014-12-12 09:51:20]

The CQ bit was checked by sdefresne@chromium.org

[commit-bot: I haz the power, 2014-12-12 09:52:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/771723002/80001

[commit-bot: I haz the power, 2014-12-12 14:36:11]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2014-12-12 14:36:48]

Patchset 5 (id:??) landed as
https://crrev.com/987f84d0d84d4e59b36ce8532ddcaf174f1346cd
Cr-Commit-Position: refs/heads/master@{#308069}