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 4cf7f23eaeb0cae45c9583eb7863b5741b8e0a9b..82d81b0e755f24daa532bad1ebb63c1248ea292d 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 alphaOption() const {
+    return m_premultiplyAlpha ? AlphaPremultiplied : AlphaNotPremultiplied;

[scroggo_chromium, 2017/04/07 18:06:06]

I wonder if we should store an AlphaOption instead of a boolean. Seems a little
simpler (sometimes we end up converting the bool back to an AlphaOption; OTOH
sometimes we do want the bool...).

[msarett1, 2017/04/10 14:42:46]

I think it makes more sense to store the AlphaOption, but I don't want to make
that part of this CL.

[scroggo_chromium, 2017/04/10 17:31:23]

sgtm

+  }
+
   // 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