Checking for valid colorType, alphaType, colorSpace in SkCodec

src/android/SkBitmapRegionCodec.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkAndroidCodec.h"
 #include "SkBitmapRegionCodec.h"
 #include "SkBitmapRegionDecoderPriv.h"
@@ skipping 117 matching lines @@
 
     // Intialize the color table
     if (kIndex_8_SkColorType == dstColorType) {
         colorTable->dangerous_overwriteColors(colors, maxColors);
     }
 
     return true;
 }
 
 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());
 }