diff options
| author |  Matt Sarett <msarett@google.com> | 2016-12-19 14:33:35 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-12-19 20:23:03 +0000 |
| commit | 2410717f900c2691db880d84a2e03a6a24905ee2 (patch) | |
| tree | c89ffb3c18cc289fc68f37e166235dc36456b74b /tests/ColorSpaceTest.cpp | |
| parent | 65869fb64b56a4c59d74003c1fac5dffc8a8bf65 (diff) | |
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>
Diffstat (limited to 'tests/ColorSpaceTest.cpp')
| -rw-r--r-- | tests/ColorSpaceTest.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/ColorSpaceTest.cpp b/tests/ColorSpaceTest.cpp index e1ff1609b2..070d50264f 100644 --- a/tests/ColorSpaceTest.cpp +++ b/tests/ColorSpaceTest.cpp @@ -240,9 +240,9 @@ DEF_TEST(ColorSpace_Serialize, r) { SkColorSpaceTransferFn fn; fn.fA = 1.0f; fn.fB = 0.0f; - fn.fC = 0.0f; + fn.fC = 1.0f; fn.fD = 0.5f; - fn.fE = 1.0f; + fn.fE = 0.0f; fn.fF = 0.0f; fn.fG = 1.0f; SkMatrix44 toXYZ(SkMatrix44::kIdentity_Constructor); @@ -265,9 +265,9 @@ DEF_TEST(ColorSpace_Equals, r) { SkColorSpaceTransferFn fn; fn.fA = 1.0f; fn.fB = 0.0f; - fn.fC = 0.0f; + fn.fC = 1.0f; fn.fD = 0.5f; - fn.fE = 1.0f; + fn.fE = 0.0f; fn.fF = 0.0f; fn.fG = 1.0f; SkMatrix44 toXYZ(SkMatrix44::kIdentity_Constructor); |