Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1229)

Side by Side Diff: tests/ColorSpaceTest.cpp

Issue 2389983002: Refactored SkColorSpace and added in a Lab PCS GM (Closed)
Patch Set: migrated call from SkColorSpace_Base::makeLinearGamma() to SkColorSpace_XYZ::makeLinearGamma() Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tests/CodecTest.cpp ('k') | tests/ColorSpaceXformTest.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2016 Google Inc. 2 * Copyright 2016 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "Resources.h" 8 #include "Resources.h"
9 #include "SkCodec.h" 9 #include "SkCodec.h"
10 #include "SkColorSpace.h" 10 #include "SkColorSpace.h"
11 #include "SkColorSpace_Base.h" 11 #include "SkColorSpace_Base.h"
12 #include "SkColorSpace_XYZ.h"
12 #include "Test.h" 13 #include "Test.h"
13 14
14 #include "png.h" 15 #include "png.h"
15 16
16 static bool almost_equal(float a, float b) { 17 static bool almost_equal(float a, float b) {
17 return SkTAbs(a - b) < 0.001f; 18 return SkTAbs(a - b) < 0.001f;
18 } 19 }
19 20
20 static void test_space(skiatest::Reporter* r, SkColorSpace* space, 21 static void test_space(skiatest::Reporter* r, SkColorSpace* space,
21 const float red[], const float green[], const float blue[ ], 22 const float red[], const float green[], const float blue[ ],
22 const SkGammaNamed expectedGamma) { 23 const SkGammaNamed expectedGamma) {
23 24
24 REPORTER_ASSERT(r, nullptr != space); 25 REPORTER_ASSERT(r, nullptr != space);
25 REPORTER_ASSERT(r, expectedGamma == as_CSB(space)->gammaNamed()); 26 SkASSERT(SkColorSpace_Base::Type::kXYZ == as_CSB(space)->type());
27 SkColorSpace_XYZ* csXYZ = static_cast<SkColorSpace_XYZ*>(space);
28 REPORTER_ASSERT(r, expectedGamma == csXYZ->gammaNamed());
26 29
27 const SkMatrix44& mat = as_CSB(space)->toXYZD50(); 30 const SkMatrix44& mat = *csXYZ->toXYZD50();
28 const float src[] = { 31 const float src[] = {
29 1, 0, 0, 1, 32 1, 0, 0, 1,
30 0, 1, 0, 1, 33 0, 1, 0, 1,
31 0, 0, 1, 1, 34 0, 0, 1, 1,
32 }; 35 };
33 const float* ref[3] = { red, green, blue }; 36 const float* ref[3] = { red, green, blue };
34 float dst[4]; 37 float dst[4];
35 for (int i = 0; i < 3; ++i) { 38 for (int i = 0; i < 3; ++i) {
36 mat.mapScalars(&src[i*4], dst); 39 mat.mapScalars(&src[i*4], dst);
37 REPORTER_ASSERT(r, almost_equal(ref[i][0], dst[0])); 40 REPORTER_ASSERT(r, almost_equal(ref[i][0], dst[0]));
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 sk_sp<SkColorSpace> strangeColorSpace = 116 sk_sp<SkColorSpace> strangeColorSpace =
114 SkColorSpace::NewRGB(SkColorSpace::kSRGB_RenderTargetGamma, srgbToxy zD50); 117 SkColorSpace::NewRGB(SkColorSpace::kSRGB_RenderTargetGamma, srgbToxy zD50);
115 REPORTER_ASSERT(r, strangeColorSpace != namedColorSpace); 118 REPORTER_ASSERT(r, strangeColorSpace != namedColorSpace);
116 } 119 }
117 120
118 DEF_TEST(ColorSpaceSRGBLinearCompare, r) { 121 DEF_TEST(ColorSpaceSRGBLinearCompare, r) {
119 // Create the linear sRGB color space by name 122 // Create the linear sRGB color space by name
120 sk_sp<SkColorSpace> namedColorSpace = SkColorSpace::NewNamed(SkColorSpace::k SRGBLinear_Named); 123 sk_sp<SkColorSpace> namedColorSpace = SkColorSpace::NewNamed(SkColorSpace::k SRGBLinear_Named);
121 124
122 // Create the linear sRGB color space via the sRGB color space's makeLinearG amma() 125 // Create the linear sRGB color space via the sRGB color space's makeLinearG amma()
123 sk_sp<SkColorSpace> viaSrgbColorSpace = 126 auto srgb = SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named);
124 as_CSB(SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named))->makeLinearGam ma(); 127 auto srgbXYZ = static_cast<SkColorSpace_XYZ*>(srgb.get());
128 sk_sp<SkColorSpace> viaSrgbColorSpace = srgbXYZ->makeLinearGamma();
125 REPORTER_ASSERT(r, namedColorSpace == viaSrgbColorSpace); 129 REPORTER_ASSERT(r, namedColorSpace == viaSrgbColorSpace);
126 130
127 // Create a linear sRGB color space by value 131 // Create a linear sRGB color space by value
128 SkMatrix44 srgbToxyzD50(SkMatrix44::kUninitialized_Constructor); 132 SkMatrix44 srgbToxyzD50(SkMatrix44::kUninitialized_Constructor);
129 srgbToxyzD50.set3x3RowMajorf(g_sRGB_XYZ); 133 srgbToxyzD50.set3x3RowMajorf(g_sRGB_XYZ);
130 sk_sp<SkColorSpace> rgbColorSpace = 134 sk_sp<SkColorSpace> rgbColorSpace =
131 SkColorSpace::NewRGB(SkColorSpace::kLinear_RenderTargetGamma, srgbToxyzD 50); 135 SkColorSpace::NewRGB(SkColorSpace::kLinear_RenderTargetGamma, srgbToxyzD 50);
132 REPORTER_ASSERT(r, rgbColorSpace == namedColorSpace); 136 REPORTER_ASSERT(r, rgbColorSpace == namedColorSpace);
133 137
134 // Change a single value from the sRGB matrix 138 // Change a single value from the sRGB matrix
(...skipping 24 matching lines...) Expand all
159 GetResourcePath("icc_profiles/HP_ZR30w.icc").c_str()); 163 GetResourcePath("icc_profiles/HP_ZR30w.icc").c_str());
160 REPORTER_ASSERT(r, monitorData); 164 REPORTER_ASSERT(r, monitorData);
161 if (!monitorData) { 165 if (!monitorData) {
162 return; 166 return;
163 } 167 }
164 sk_sp<SkColorSpace> monitorSpace = SkColorSpace::NewICC(monitorData->data(), 168 sk_sp<SkColorSpace> monitorSpace = SkColorSpace::NewICC(monitorData->data(),
165 monitorData->size()) ; 169 monitorData->size()) ;
166 sk_sp<SkData> newMonitorData = ColorSpaceTest::WriteToICC(monitorSpace.get() ); 170 sk_sp<SkData> newMonitorData = ColorSpaceTest::WriteToICC(monitorSpace.get() );
167 sk_sp<SkColorSpace> newMonitorSpace = SkColorSpace::NewICC(newMonitorData->d ata(), 171 sk_sp<SkColorSpace> newMonitorSpace = SkColorSpace::NewICC(newMonitorData->d ata(),
168 newMonitorData->s ize()); 172 newMonitorData->s ize());
169 REPORTER_ASSERT(r, as_CSB(monitorSpace)->toXYZD50() == as_CSB(newMonitorSpac e)->toXYZD50()); 173 SkASSERT(SkColorSpace_Base::Type::kXYZ == as_CSB(monitorSpace)->type());
170 REPORTER_ASSERT(r, as_CSB(monitorSpace)->gammaNamed() == as_CSB(newMonitorSp ace)->gammaNamed()); 174 SkColorSpace_XYZ* monitorSpaceXYZ = static_cast<SkColorSpace_XYZ*>(monitorSp ace.get());
175 SkASSERT(SkColorSpace_Base::Type::kXYZ == as_CSB(newMonitorSpace)->type());
176 SkColorSpace_XYZ* newMonitorSpaceXYZ = static_cast<SkColorSpace_XYZ*>(newMon itorSpace.get());
177 REPORTER_ASSERT(r, *monitorSpaceXYZ->toXYZD50() == *newMonitorSpaceXYZ->toXY ZD50());
178 REPORTER_ASSERT(r, monitorSpaceXYZ->gammaNamed() == newMonitorSpaceXYZ->gamm aNamed());
171 } 179 }
172 180
173 DEF_TEST(ColorSpace_Named, r) { 181 DEF_TEST(ColorSpace_Named, r) {
174 const struct { 182 const struct {
175 SkColorSpace::Named fNamed; 183 SkColorSpace::Named fNamed;
176 SkGammaNamed fExpectedGamma; 184 SkGammaNamed fExpectedGamma;
177 } recs[] { 185 } recs[] {
178 { SkColorSpace::kSRGB_Named, kSRGB_SkGammaNamed }, 186 { SkColorSpace::kSRGB_Named, kSRGB_SkGammaNamed },
179 { SkColorSpace::kAdobeRGB_Named, k2Dot2Curve_SkGammaNamed }, 187 { SkColorSpace::kAdobeRGB_Named, k2Dot2Curve_SkGammaNamed },
180 { SkColorSpace::kSRGBLinear_Named, kLinear_SkGammaNamed }, 188 { SkColorSpace::kSRGBLinear_Named, kLinear_SkGammaNamed },
181 }; 189 };
182 190
183 for (auto rec : recs) { 191 for (auto rec : recs) {
184 auto cs = SkColorSpace::NewNamed(rec.fNamed); 192 auto cs = SkColorSpace::NewNamed(rec.fNamed);
185 REPORTER_ASSERT(r, cs); 193 REPORTER_ASSERT(r, cs);
186 if (cs) { 194 if (cs) {
187 REPORTER_ASSERT(r, rec.fExpectedGamma == as_CSB(cs)->gammaNamed()); 195 SkASSERT(SkColorSpace_Base::Type::kXYZ == as_CSB(cs)->type());
196 SkColorSpace_XYZ* csXYZ = static_cast<SkColorSpace_XYZ*>(cs.get());
197 REPORTER_ASSERT(r, rec.fExpectedGamma == csXYZ->gammaNamed());
188 } 198 }
189 } 199 }
190 200
191 SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType); 201 SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType);
192 REPORTER_ASSERT(r, info.gammaCloseToSRGB()); 202 REPORTER_ASSERT(r, info.gammaCloseToSRGB());
193 } 203 }
194 204
195 static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNa med) { 205 static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNa med) {
196 sk_sp<SkData> data1 = space->serialize(); 206 sk_sp<SkData> data1 = space->serialize();
197 207
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
306 primaries.fWX = 0.3127f; 316 primaries.fWX = 0.3127f;
307 primaries.fWY = 0.3290f; 317 primaries.fWY = 0.3290f;
308 318
309 SkMatrix44 toXYZ(SkMatrix44::kUninitialized_Constructor); 319 SkMatrix44 toXYZ(SkMatrix44::kUninitialized_Constructor);
310 bool result = primaries.toXYZD50(&toXYZ); 320 bool result = primaries.toXYZD50(&toXYZ);
311 REPORTER_ASSERT(r, result); 321 REPORTER_ASSERT(r, result);
312 322
313 sk_sp<SkColorSpace> space = SkColorSpace::NewRGB(SkColorSpace::kSRGB_RenderT argetGamma, toXYZ); 323 sk_sp<SkColorSpace> space = SkColorSpace::NewRGB(SkColorSpace::kSRGB_RenderT argetGamma, toXYZ);
314 REPORTER_ASSERT(r, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named) == spac e); 324 REPORTER_ASSERT(r, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named) == spac e);
315 } 325 }
OLDNEW
« no previous file with comments | « tests/CodecTest.cpp ('k') | tests/ColorSpaceXformTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698