diff options
author | 2016-08-14 15:31:28 -0700 | |
---|---|---|
committer | 2016-08-14 15:31:28 -0700 | |
commit | de4422203375b5fdbf192bb2f2576cf807698d61 (patch) | |
tree | ead2fc5ffd45a2d51adff8c8ca02c0c7928f8370 | |
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
-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; |