Chromium Code Reviews Chromium Code Reviews
Issue 1641273003:
Support decoding opaque to *premul (Closed)
Base URL: https://skia.googlesource.com/skia.git@master| Index: src/codec/SkMaskSwizzler.cpp |
| diff --git a/src/codec/SkMaskSwizzler.cpp b/src/codec/SkMaskSwizzler.cpp |
| index 01502cbd4ab875ac3ca24e26a61ff85288556423..1b77a85b139c591736e9edd5e6f152b11d6e232f 100644 |
| --- a/src/codec/SkMaskSwizzler.cpp |
| +++ b/src/codec/SkMaskSwizzler.cpp |
| @@ -250,13 +250,7 @@ SkMaskSwizzler* SkMaskSwizzler::CreateMaskSwizzler(const SkImageInfo& dstInfo, |
| } |
| break; |
| case kRGB_565_SkColorType: |
| - switch (dstInfo.alphaType()) { |
| - case kOpaque_SkAlphaType: |
| - proc = &swizzle_mask16_to_565; |
| - break; |
| - default: |
| - break; |
| - } |
| + proc = &swizzle_mask16_to_565; |
|
scroggo
2016/02/03 14:58:45
These changes fix the crash I was seeing on the tr
msarett
2016/02/03 15:03:07
Acknowledged.
|
| break; |
| default: |
| break; |
| @@ -280,13 +274,7 @@ SkMaskSwizzler* SkMaskSwizzler::CreateMaskSwizzler(const SkImageInfo& dstInfo, |
| } |
| break; |
| case kRGB_565_SkColorType: |
| - switch (dstInfo.alphaType()) { |
| - case kOpaque_SkAlphaType: |
| - proc = &swizzle_mask24_to_565; |
| - break; |
| - default: |
| - break; |
| - } |
| + proc = &swizzle_mask24_to_565; |
| break; |
| default: |
| break; |
| @@ -310,13 +298,7 @@ SkMaskSwizzler* SkMaskSwizzler::CreateMaskSwizzler(const SkImageInfo& dstInfo, |
| } |
| break; |
| case kRGB_565_SkColorType: |
| - switch (dstInfo.alphaType()) { |
| - case kOpaque_SkAlphaType: |
| - proc = &swizzle_mask32_to_565; |
| - break; |
| - default: |
| - break; |
| - } |
| + proc = &swizzle_mask32_to_565; |
| break; |
| default: |
| break; |
