diff options
author | msarett <msarett@google.com> | 2015-04-09 12:43:10 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-09 12:43:10 -0700 |
commit | 438b2adefb9e9213e0ddaf0609405d3087a1cf0a (patch) | |
tree | 34b6c116c074d5f24f8e37a45db0e8404de8d83f /dm/DMSrcSink.cpp | |
parent | 16b254a200f63e85041cac9a283ff0ff14d94ba1 (diff) |
***Disables swizzles to 565.
We may want to enable swizzles to 565
for images that are encoded in a format
similar to 565, however, we do not want
to take images that decode naturally to
kN32 and then convert them to 565.
***Enable swizzles to kIndex_8. For images
encoded in a color table format, we suggest
that they be decoded to kIndex_8. When we
decode, we only allow conversion to kIndex_8
if it matches the suggested color type (except
wbmp which seems good as is).
***Modify dm to test images that decode to
kIndex_8.
BUG=skia:3257
BUG=skia:3440
Review URL: https://codereview.chromium.org/1055743003
Diffstat (limited to 'dm/DMSrcSink.cpp')
-rw-r--r-- | dm/DMSrcSink.cpp | 46 |
1 files changed, 38 insertions, 8 deletions
diff --git a/dm/DMSrcSink.cpp b/dm/DMSrcSink.cpp index de9ee76c80..9e9a77c271 100644 --- a/dm/DMSrcSink.cpp +++ b/dm/DMSrcSink.cpp @@ -52,7 +52,11 @@ Name GMSrc::name() const { /*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~*/ -CodecSrc::CodecSrc(Path path, Mode mode) : fPath(path), fMode(mode) {} +CodecSrc::CodecSrc(Path path, Mode mode, DstColorType dstColorType) + : fPath(path) + , fMode(mode) + , fDstColorType(dstColorType) +{} Error CodecSrc::draw(SkCanvas* canvas) const { SkImageInfo canvasInfo; @@ -66,27 +70,53 @@ Error CodecSrc::draw(SkCanvas* canvas) const { if (!encoded) { return SkStringPrintf("Couldn't read %s.", fPath.c_str()); } - SkAutoTDelete<SkCodec> codec(SkCodec::NewFromData(encoded)); - if (!codec) { - return SkStringPrintf("Couldn't decode %s.", fPath.c_str()); + if (NULL == codec.get()) { + return SkStringPrintf("Couldn't create codec for %s.", fPath.c_str()); + } + + // Choose the color type to decode to + SkImageInfo decodeInfo = codec->getInfo(); + SkColorType canvasColorType = canvasInfo.colorType(); + switch (fDstColorType) { + case kIndex8_Always_DstColorType: + case kGrayscale_Always_DstColorType: + if (kRGB_565_SkColorType == canvasColorType) { + return Error::Nonfatal("Testing non-565 to 565 is uninteresting."); + } + break; + default: + decodeInfo = decodeInfo.makeColorType(canvasColorType); + break; + } + + // Construct a color table for the decode if necessary + SkAutoTUnref<SkColorTable> colorTable(NULL); + SkPMColor* colorPtr = NULL; + int* colorCountPtr = NULL; + int maxColors = 256; + if (kIndex_8_SkColorType == decodeInfo.colorType()) { + SkPMColor colors[256]; + colorTable.reset(SkNEW_ARGS(SkColorTable, (colors, maxColors))); + colorPtr = const_cast<SkPMColor*>(colorTable->readColors()); + colorCountPtr = &maxColors; } - SkImageInfo decodeInfo = codec->getInfo().makeColorType(canvasInfo.colorType()); + // FIXME: Currently we cannot draw unpremultiplied sources. if (decodeInfo.alphaType() == kUnpremul_SkAlphaType) { - // FIXME: Currently we cannot draw unpremultiplied sources. decodeInfo = decodeInfo.makeAlphaType(kPremul_SkAlphaType); } SkBitmap bitmap; - if (!bitmap.tryAllocPixels(decodeInfo)) { + if (!bitmap.tryAllocPixels(decodeInfo, NULL, colorTable.get())) { return SkStringPrintf("Image(%s) is too large (%d x %d)\n", fPath.c_str(), decodeInfo.width(), decodeInfo.height()); } switch (fMode) { case kNormal_Mode: - switch (codec->getPixels(decodeInfo, bitmap.getPixels(), bitmap.rowBytes())) { + switch (codec->getPixels(decodeInfo, bitmap.getPixels(), bitmap.rowBytes(), NULL, + colorPtr, colorCountPtr)) { case SkImageGenerator::kSuccess: // We consider incomplete to be valid, since we should still decode what is // available. |