cc: Use SkCreateColorSpaceXformCanvas for color transforms

cc/playback/raster_source.h

Index: cc/playback/raster_source.h
diff --git a/cc/playback/raster_source.h b/cc/playback/raster_source.h
index 522557cbb8daa2b77975177f965a412eafa0d0ee..820d78de01b213449d14a0b600c9376a4e3ed3bf 100644
--- a/cc/playback/raster_source.h
+++ b/cc/playback/raster_source.h
@@ -17,6 +17,7 @@
 #include "cc/playback/recording_source.h"
 #include "skia/ext/analysis_canvas.h"
 #include "third_party/skia/include/core/SkPicture.h"
+#include "ui/gfx/color_space.h"
 
 namespace cc {
 class DisplayItemList;
@@ -42,6 +43,10 @@ class CC_EXPORT RasterSource : public base::RefCountedThreadSafe<RasterSource> {
     // compositor image caching.
     bool use_image_hijack_canvas;
 
+    // If set, use a SkColorSpaceXformCanvas to specify a transform all inputs
+    // to this color space.
+    gfx::ColorSpace target_color_space;

[enne (OOO), 2017/03/15 22:15:40]

Maybe I'm missing something, but would it make sense to hoist this up to the
caller of RasterSource that passes in the SkCanvas to just pass in the color
space canvas? The RasterSource doesn't have a color space itself, it's rastered
onto a canvas with a target color space, right?

[ccameron, 2017/03/16 05:11:34]

That makes a lot of sense -- we can then the color space directly from the
resource. I've re-shifted things to do this.

+
     // If non-empty, an image hijack canvas will be used to skip these images
     // during raster.
     // TODO(khushalsagar): Consolidate more settings for playback here? See
@@ -151,7 +156,9 @@ class CC_EXPORT RasterSource : public base::RefCountedThreadSafe<RasterSource> {
   ImageDecodeCache* image_decode_cache_;
 
  private:
-  void RasterCommon(SkCanvas* canvas, SkPicture::AbortCallback* callback) const;
+  void RasterCommon(SkCanvas* canvas,
+                    const gfx::ColorSpace& target_color_space,
+                    SkPicture::AbortCallback* callback) const;
 
   void PrepareForPlaybackToCanvas(SkCanvas* canvas) const;