diff options
author | msarett <msarett@google.com> | 2016-01-19 10:40:12 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-19 10:40:12 -0800 |
commit | bda86098ab2784968d0a1222dd1f4c18d18fe050 (patch) | |
tree | 7e81abd15acc0186a95ec3454f85dc663c36a996 /src/codec/SkJpegCodec.cpp | |
parent | 94ec356ed4d8ed0cef39a43b221bee9394f74a8e (diff) |
Clean up unused or overly complex modes in SkSwizzler
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1582083005
Review URL: https://codereview.chromium.org/1582083005
Diffstat (limited to 'src/codec/SkJpegCodec.cpp')
-rw-r--r-- | src/codec/SkJpegCodec.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/src/codec/SkJpegCodec.cpp b/src/codec/SkJpegCodec.cpp index 7db772da63..89925fb464 100644 --- a/src/codec/SkJpegCodec.cpp +++ b/src/codec/SkJpegCodec.cpp @@ -329,18 +329,17 @@ void SkJpegCodec::initializeSwizzler(const SkImageInfo& dstInfo, const Options& if (JCS_CMYK == fDecoderMgr->dinfo()->out_color_space) { srcConfig = SkSwizzler::kCMYK; } else { + // If the out_color_space is not CMYK, the only reason we would need a swizzler is + // for sampling and/or subsetting. switch (dstInfo.colorType()) { case kGray_8_SkColorType: - srcConfig = SkSwizzler::kGray; + srcConfig = SkSwizzler::kNoOp8; break; - case kRGBA_8888_SkColorType: - srcConfig = SkSwizzler::kRGBX; - break; - case kBGRA_8888_SkColorType: - srcConfig = SkSwizzler::kBGRX; + case kN32_SkColorType: + srcConfig = SkSwizzler::kNoOp32; break; case kRGB_565_SkColorType: - srcConfig = SkSwizzler::kRGB_565; + srcConfig = SkSwizzler::kNoOp16; break; default: // This function should only be called if the colorType is supported by jpeg |