Introduce SkGammas type to represent ICC gamma curves

tests/ColorSpaceTest.cpp

Index: tests/ColorSpaceTest.cpp
diff --git a/tests/ColorSpaceTest.cpp b/tests/ColorSpaceTest.cpp
index 313eed86f118a9b9632f4c514baa5e21813dc801..859d02dd93fdd277733fc452b6b6ec1cc5da470d 100644
--- a/tests/ColorSpaceTest.cpp
+++ b/tests/ColorSpaceTest.cpp
@@ -35,12 +35,14 @@ DEF_TEST(ColorSpaceParsePngICCProfile, r) {
     SkColorSpace* colorSpace = codec->getColorSpace();
     REPORTER_ASSERT(r, nullptr != colorSpace);
 
-    // No need to use almost equal here.  The color profile that we have extracted
-    // actually has a table of gammas.  And our current implementation guesses 2.2f.
-    SkFloat3 gammas = colorSpace->gamma();
-    REPORTER_ASSERT(r, 2.2f == gammas.fVec[0]);
-    REPORTER_ASSERT(r, 2.2f == gammas.fVec[1]);
-    REPORTER_ASSERT(r, 2.2f == gammas.fVec[2]);
+    // The color profile that we have extracted has represents gamma with a lookup table.
+    // So we expect the gamma value to be zero.
+    const SkColorSpace::SkGammas& gammas = colorSpace->gammas();

[scroggo, 2016/04/29 15:29:26]

put gammas inside the SK_DEBUG code?

[msarett, 2016/04/29 15:44:32]

Done.

+#ifdef SK_DEBUG
+    REPORTER_ASSERT(r, 0.0f == gammas.red());
+    REPORTER_ASSERT(r, 0.0f == gammas.green());
+    REPORTER_ASSERT(r, 0.0f == gammas.blue());
+#endif
 
     // These nine values were extracted from the color profile in isolation (before
     // we embedded it in the png).  Here we check that we still extract the same values.
@@ -75,10 +77,12 @@ DEF_TEST(ColorSpaceParseJpegICCProfile, r) {
 
     // It's important to use almost equal here.  This profile sets gamma as
     // 563 / 256, which actually comes out to about 2.19922.
-    SkFloat3 gammas = colorSpace->gamma();
-    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.fVec[0]));
-    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.fVec[1]));
-    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.fVec[2]));
+    const SkColorSpace::SkGammas& gammas = colorSpace->gammas();
+#ifdef SK_DEBUG
+    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.red()));
+    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.green()));
+    REPORTER_ASSERT(r, almost_equal(2.2f, gammas.blue()));
+#endif
 
     // These nine values were extracted from the color profile.  Until we know any
     // better, we'll assume these are the right values and test that we continue