Make BitmapRegionDecoder succeed on invalid requests

include/codec/SkAndroidCodec.h

Index: include/codec/SkAndroidCodec.h
diff --git a/include/codec/SkAndroidCodec.h b/include/codec/SkAndroidCodec.h
index f979886a43ec33c573de3553b49651a8a06d7025..64880076a37a44f7042157d65f77777043c247f9 100644
--- a/include/codec/SkAndroidCodec.h
+++ b/include/codec/SkAndroidCodec.h
@@ -53,6 +53,23 @@ public:
     SkEncodedFormat getEncodedFormat() const { return this->onGetEncodedFormat(); }
 
     /**
+     *  @param requestedColorType Color type requested by the client
+     *
+     *  @return If it is possible to decode to requestedColorType, this returns
+     *          requestedColorType.  Otherwise, this returns kN32_SkColorType.
+     */
+    SkColorType fixRequestedColorType(SkColorType requestedColorType);

[djsollen, 2015/12/10 21:06:45]

this seems like an odd name to me.  something like compute, find, or select seem
like a better fit than fixRequested.

[msarett, 2015/12/10 22:41:55]

Changing to computeOutput*Type()

+
+    /**
+     *  @param requestedUnpremul  Indicates if the client requested
+     *                            unpremultiplied output
+     *
+     *  @return Returns the appropriate alpha type to decode to.  If the image
+     *          has alpha, the value of requestedUnpremul will be honored.
+     */
+    SkAlphaType fixRequestedAlphaType(bool requestedUnpremul);
+
+    /**
      *  Returns the dimensions of the scaled output image, for an input
      *  sampleSize.
      *