commit 2410717f900c2691db880d84a2e03a6a24905ee2
author Matt Sarett <msarett@google.com> Mon Dec 19 14:33:35 2016 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Dec 19 20:23:03 2016 +0000
tree c89ffb3c18cc289fc68f37e166235dc36456b74b
parent 65869fb64b56a4c59d74003c1fac5dffc8a8bf65

Fix swapped interpretation of c and e in SkColorSpace_ICC

The ICC errata supports the opposite of what we do.
http://www.color.org/icc_specs2.xalter

TBR=reed@google.com

BUG=skia:

CQ_INCLUDE_TRYBOTS=skia.primary:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD

Change-Id: I18ace7f312926b264e624c30d8cb983eff5c434b
Reviewed-on: https://skia-review.googlesource.com/6277
Commit-Queue: Matt Sarett <msarett@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>

tests/ColorSpaceXformTest.cpp

diff --git a/tests/ColorSpaceXformTest.cpp b/tests/ColorSpaceXformTest.cpp
index d2bd2a3..413d74e 100644
--- a/tests/ColorSpaceXformTest.cpp
+++ b/tests/ColorSpaceXformTest.cpp
@@ -176,18 +176,18 @@
     SkColorSpaceTransferFn* params = SkTAddOffset<SkColorSpaceTransferFn>
             (memory, sizeof(SkGammas));
 
-    // Interval, switch xforms at 0.0031308f
+    // Interval.
     params->fD = 0.04045f;
 
     // First equation:
-    params->fE = 1.0f / 12.92f;
+    params->fC = 1.0f / 12.92f;
     params->fF = 0.0f;
 
     // Second equation:
     // Note that the function is continuous (it's actually sRGB).
     params->fA = 1.0f / 1.055f;
     params->fB = 0.055f / 1.055f;
-    params->fC = 0.0f;
+    params->fE = 0.0f;
     params->fG = 2.4f;
     test_identity_xform(r, gammas, true);
     test_identity_xform_A2B(r, kNonStandard_SkGammaNamed, gammas);
@@ -239,9 +239,9 @@
             sizeof(SkGammas) + sizeof(float) * tableSize);
     params->fA = 1.0f / 1.055f;
     params->fB = 0.055f / 1.055f;
-    params->fC = 0.0f;
+    params->fC = 1.0f / 12.92f;
     params->fD = 0.04045f;
-    params->fE = 1.0f / 12.92f;
+    params->fE = 0.0f;
     params->fF = 0.0f;
     params->fG = 2.4f;