| commit | de4422203375b5fdbf192bb2f2576cf807698d61 | [log] [tgz] |
|---|---|---|
| author | msarett <msarett@google.com> | Sun Aug 14 15:31:28 2016 -0700 |
| committer | Commit bot <commit-bot@chromium.org> | Sun Aug 14 15:31:28 2016 -0700 |
| 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
diff --git a/src/codec/SkWebpCodec.cpp b/src/codec/SkWebpCodec.cpp index cefde2d..e434142 100644 --- a/src/codec/SkWebpCodec.cpp +++ b/src/codec/SkWebpCodec.cpp
@@ -90,9 +90,6 @@ 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;