Simplify color space xform storage and usage (float[] vs. SkMatrix44)

include/gpu/GrColorSpaceXform.h

Index: include/gpu/GrColorSpaceXform.h
diff --git a/include/gpu/GrColorSpaceXform.h b/include/gpu/GrColorSpaceXform.h
index ea90739fd2f0354f931de32c5b5112db4163aa5a..8450190e544b90e89fda05360bac0618c5b06c5a 100644
--- a/include/gpu/GrColorSpaceXform.h
+++ b/include/gpu/GrColorSpaceXform.h
@@ -8,24 +8,24 @@
 #ifndef GrColorSpaceXform_DEFINED
 #define GrColorSpaceXform_DEFINED
 
-#include "SkMatrix44.h"
 #include "SkRefCnt.h"
 
 class SkColorSpace;
+class SkMatrix44;
 
  /**
   * Represents a color gamut transformation (as a 4x4 color matrix)
   */
 class GrColorSpaceXform : public SkRefCnt {
 public:
-    GrColorSpaceXform(const SkMatrix44& srcToDst) : fSrcToDst(srcToDst) {}
+    GrColorSpaceXform(const SkMatrix44& srcToDst);
 
     static sk_sp<GrColorSpaceXform> Make(SkColorSpace* src, SkColorSpace* dst);
 
-    const SkMatrix44& srcToDst() { return fSrcToDst; }
+    const float* srcToDst() { return fSrcToDst; }
 
 private:

[robertphillips, 2016/07/26 19:34:38]

// We store the column major form of the original srcToDst matrix ...

?

[Brian Osman, 2016/07/26 23:14:58]

Done.

-    SkMatrix44 fSrcToDst;
+    float fSrcToDst[16];
 };
 
 #endif