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

Side by Side Diff: tests/ColorSpaceTest.cpp

Issue 2389983002: Refactored SkColorSpace and added in a Lab PCS GM (Closed)
Patch Set: multiplied the matrix for the temporary XYZTRC profile from an A2B0 profile code to match what woul… 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
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_XYZTRC.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 REPORTER_ASSERT(r, expectedGamma == as_CSXYZ(space)->gammaNamed());
26 27
27 const SkMatrix44& mat = as_CSB(space)->toXYZD50(); 28 const SkMatrix44& mat = as_CSXYZ(space)->toXYZD50();
28 const float src[] = { 29 const float src[] = {
29 1, 0, 0, 1, 30 1, 0, 0, 1,
30 0, 1, 0, 1, 31 0, 1, 0, 1,
31 0, 0, 1, 1, 32 0, 0, 1, 1,
32 }; 33 };
33 const float* ref[3] = { red, green, blue }; 34 const float* ref[3] = { red, green, blue };
34 float dst[4]; 35 float dst[4];
35 for (int i = 0; i < 3; ++i) { 36 for (int i = 0; i < 3; ++i) {
36 mat.mapScalars(&src[i*4], dst); 37 mat.mapScalars(&src[i*4], dst);
37 REPORTER_ASSERT(r, almost_equal(ref[i][0], dst[0])); 38 REPORTER_ASSERT(r, almost_equal(ref[i][0], dst[0]));
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 GetResourcePath("icc_profiles/HP_ZR30w.icc").c_str()); 160 GetResourcePath("icc_profiles/HP_ZR30w.icc").c_str());
160 REPORTER_ASSERT(r, monitorData); 161 REPORTER_ASSERT(r, monitorData);
161 if (!monitorData) { 162 if (!monitorData) {
162 return; 163 return;
163 } 164 }
164 sk_sp<SkColorSpace> monitorSpace = SkColorSpace::NewICC(monitorData->data(), 165 sk_sp<SkColorSpace> monitorSpace = SkColorSpace::NewICC(monitorData->data(),
165 monitorData->size()) ; 166 monitorData->size()) ;
166 sk_sp<SkData> newMonitorData = ColorSpaceTest::WriteToICC(monitorSpace.get() ); 167 sk_sp<SkData> newMonitorData = ColorSpaceTest::WriteToICC(monitorSpace.get() );
167 sk_sp<SkColorSpace> newMonitorSpace = SkColorSpace::NewICC(newMonitorData->d ata(), 168 sk_sp<SkColorSpace> newMonitorSpace = SkColorSpace::NewICC(newMonitorData->d ata(),
168 newMonitorData->s ize()); 169 newMonitorData->s ize());
169 REPORTER_ASSERT(r, as_CSB(monitorSpace)->toXYZD50() == as_CSB(newMonitorSpac e)->toXYZD50()); 170 REPORTER_ASSERT(r, as_CSXYZ(monitorSpace)->toXYZD50() == as_CSXYZ(newMonitor Space)->toXYZD50());
170 REPORTER_ASSERT(r, as_CSB(monitorSpace)->gammaNamed() == as_CSB(newMonitorSp ace)->gammaNamed()); 171 REPORTER_ASSERT(r, as_CSXYZ(monitorSpace)->gammaNamed() == as_CSXYZ(newMonit orSpace)->gammaNamed());
171 } 172 }
172 173
173 DEF_TEST(ColorSpace_Named, r) { 174 DEF_TEST(ColorSpace_Named, r) {
174 const struct { 175 const struct {
175 SkColorSpace::Named fNamed; 176 SkColorSpace::Named fNamed;
176 SkGammaNamed fExpectedGamma; 177 SkGammaNamed fExpectedGamma;
177 } recs[] { 178 } recs[] {
178 { SkColorSpace::kSRGB_Named, kSRGB_SkGammaNamed }, 179 { SkColorSpace::kSRGB_Named, kSRGB_SkGammaNamed },
179 { SkColorSpace::kAdobeRGB_Named, k2Dot2Curve_SkGammaNamed }, 180 { SkColorSpace::kAdobeRGB_Named, k2Dot2Curve_SkGammaNamed },
180 { SkColorSpace::kSRGBLinear_Named, kLinear_SkGammaNamed }, 181 { SkColorSpace::kSRGBLinear_Named, kLinear_SkGammaNamed },
181 }; 182 };
182 183
183 for (auto rec : recs) { 184 for (auto rec : recs) {
184 auto cs = SkColorSpace::NewNamed(rec.fNamed); 185 auto cs = SkColorSpace::NewNamed(rec.fNamed);
185 REPORTER_ASSERT(r, cs); 186 REPORTER_ASSERT(r, cs);
186 if (cs) { 187 if (cs) {
187 REPORTER_ASSERT(r, rec.fExpectedGamma == as_CSB(cs)->gammaNamed()); 188 REPORTER_ASSERT(r, rec.fExpectedGamma == as_CSXYZ(cs)->gammaNamed()) ;
188 } 189 }
189 } 190 }
190 191
191 SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType); 192 SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType);
192 REPORTER_ASSERT(r, info.gammaCloseToSRGB()); 193 REPORTER_ASSERT(r, info.gammaCloseToSRGB());
193 } 194 }
194 195
195 static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNa med) { 196 static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNa med) {
196 sk_sp<SkData> data1 = space->serialize(); 197 sk_sp<SkData> data1 = space->serialize();
197 198
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
264 REPORTER_ASSERT(r, !SkColorSpace::Equals(srgb.get(), nullptr)); 265 REPORTER_ASSERT(r, !SkColorSpace::Equals(srgb.get(), nullptr));
265 REPORTER_ASSERT(r, !SkColorSpace::Equals(adobe.get(), srgb.get())); 266 REPORTER_ASSERT(r, !SkColorSpace::Equals(adobe.get(), srgb.get()));
266 REPORTER_ASSERT(r, !SkColorSpace::Equals(z30.get(), srgb.get())); 267 REPORTER_ASSERT(r, !SkColorSpace::Equals(z30.get(), srgb.get()));
267 REPORTER_ASSERT(r, !SkColorSpace::Equals(z32.get(), z30.get())); 268 REPORTER_ASSERT(r, !SkColorSpace::Equals(z32.get(), z30.get()));
268 REPORTER_ASSERT(r, !SkColorSpace::Equals(upperLeft.get(), srgb.get())); 269 REPORTER_ASSERT(r, !SkColorSpace::Equals(upperLeft.get(), srgb.get()));
269 REPORTER_ASSERT(r, !SkColorSpace::Equals(upperLeft.get(), upperRight.get())) ; 270 REPORTER_ASSERT(r, !SkColorSpace::Equals(upperLeft.get(), upperRight.get())) ;
270 REPORTER_ASSERT(r, !SkColorSpace::Equals(z30.get(), upperRight.get())); 271 REPORTER_ASSERT(r, !SkColorSpace::Equals(z30.get(), upperRight.get()));
271 REPORTER_ASSERT(r, !SkColorSpace::Equals(upperRight.get(), adobe.get())); 272 REPORTER_ASSERT(r, !SkColorSpace::Equals(upperRight.get(), adobe.get()));
272 REPORTER_ASSERT(r, !SkColorSpace::Equals(rgb1.get(), rgb2.get())); 273 REPORTER_ASSERT(r, !SkColorSpace::Equals(rgb1.get(), rgb2.get()));
273 } 274 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698