diff options
| author |  msarett <msarett@google.com> | 2016-08-14 15:31:28 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-08-14 15:31:28 -0700 |
| commit | de4422203375b5fdbf192bb2f2576cf807698d61 (patch) | |
| tree | ead2fc5ffd45a2d51adff8c8ca02c0c7928f8370 /src/codec | |
| parent | 0c685ee188a5e500df25a34fa6edfceb4cbb795c (diff) | |
Webp has_alpha is a hint only
The has_alpha bit on webps is a hint only. We are correct to always
assume that the image may have alpha.
https://groups.google.com/a/webmproject.org/forum/#!msg/webp-discuss/2rFPKOzxuSM/uabYhmSNJAAJ
TBR=scroggo@google.com
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2243153002
Review-Url: https://codereview.chromium.org/2243153002
Diffstat (limited to 'src/codec')
| -rw-r--r-- | src/codec/SkWebpCodec.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/codec/SkWebpCodec.cpp b/src/codec/SkWebpCodec.cpp index cefde2d20d..e4341421bb 100644 --- a/src/codec/SkWebpCodec.cpp +++ b/src/codec/SkWebpCodec.cpp @@ -90,9 +90,6 @@ SkCodec* SkWebpCodec::NewFromStream(SkStream* stream) { break; case 2: // This is the lossless format (BGRA). - // FIXME: Should we check the has_alpha flag here? It looks - // like the image is encoded with an alpha channel - // regardless of whether or not the alpha flag is set. color = SkEncodedInfo::kBGRA_Color; alpha = SkEncodedInfo::kUnpremul_Alpha; break; |