Respect colorSpace in DecodingImageGenerator::onGetPixels()

third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h

Index: third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
diff --git a/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h b/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
index b2f1f058421eab9fae2450eb027688ae4a563b7b..1f2f27045ea4911a113e1a7f3007089e42a86b42 100644
--- a/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
+++ b/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.h
@@ -226,6 +226,10 @@ class PLATFORM_EXPORT ImageDecoder {
   // Transformation from embedded color space to target color space.
   SkColorSpaceXform* ColorTransform();
 
+  AlphaOption AlphaOpt() const {

[scroggo_chromium, 2017/04/11 12:37:38]

I suppose you abbreviated the name so it wouldn't conflict with the enum?

The new style guide
(https://docs.google.com/document/d/1ZQLiQKw9hCOyDDhDS4JhckhuiJmLqSbw9FLfySMyxgU/)
is not explicit about this case, but there's a suggestion that this should be
GetAlphaOption().

[msarett1, 2017/04/11 12:44:20]

sgtm, done.

+    return premultiply_alpha_ ? kAlphaPremultiplied : kAlphaNotPremultiplied;
+  }
+
   // Sets the "decode failure" flag.  For caller convenience (since so
   // many callers want to return false after calling this), returns false
   // to enable easy tailcalling.  Subclasses may override this to also