Towards deleting YUV to RGB computation redundancy

ui/gfx/color_space.h

Index: ui/gfx/color_space.h
diff --git a/ui/gfx/color_space.h b/ui/gfx/color_space.h
index fbbb78690df39aa52a701190369f4ae979912155..1f12b0a810e7c847ababf144d8e3d47bd83a8aad 100644
--- a/ui/gfx/color_space.h
+++ b/ui/gfx/color_space.h
@@ -178,6 +178,10 @@ class GFX_EXPORT ColorSpace {
   bool GetTransferFunction(SkColorSpaceTransferFn* fn) const;
   bool GetInverseTransferFunction(SkColorSpaceTransferFn* fn) const;
 
+  // For most formats, this is the RGB to YUV matrix.
+  void GetTransferMatrix(SkMatrix44* matrix) const;

[hubbe, 2017/02/02 04:51:37]

Maybe return a bool since this is not always possible?

[ccameron, 2017/02/02 17:33:52]

I was going back-and-forth on this, but in the end felt that it was more
appropriate to leave this together, since that's the structure that was already
present.

If we were to separate it out, I'd prefer that we put the full BT2020_CL
conversion in its own distinct function.

[hubbe, 2017/02/02 18:35:25]

Doing so would make the implementation a lot bigger and harder to optimize.

+  void GetRangeAdjustMatrix(SkMatrix44* matrix) const;
+
  private:
   PrimaryID primaries_ = PrimaryID::UNSPECIFIED;
   TransferID transfer_ = TransferID::UNSPECIFIED;