Checking for valid colorType, alphaType, colorSpace in SkCodec

src/android/SkBitmapRegionCodec.cpp

Index: src/android/SkBitmapRegionCodec.cpp
diff --git a/src/android/SkBitmapRegionCodec.cpp b/src/android/SkBitmapRegionCodec.cpp
index 9c21484ec55294626913aa34183a3d1fa25ae588..063182f4761e85cccad275655ca7da55ae650638 100644
--- a/src/android/SkBitmapRegionCodec.cpp
+++ b/src/android/SkBitmapRegionCodec.cpp
@@ -135,5 +135,8 @@ bool SkBitmapRegionCodec::decodeRegion(SkBitmap* bitmap, SkBRDAllocator* allocat
 }
 
 bool SkBitmapRegionCodec::conversionSupported(SkColorType colorType) {
-    return conversion_possible(fCodec->getInfo().makeColorType(colorType), fCodec->getInfo());
+    // Enable legacy behavior.
+    sk_sp<SkColorSpace> colorSpace = nullptr;

[scroggo, 2016/09/08 16:33:11]

This looks to be unused

[msarett, 2016/09/08 16:49:45]

Ahh thanks.

+    SkImageInfo dstInfo = fCodec->getInfo().makeColorType(colorType).makeColorSpace(nullptr);
+    return legacy_conversion_possible(dstInfo, fCodec->getInfo());
 }