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

Side by Side Diff: tests/ColorSpaceTest.cpp

Issue 2192903002: Add writeToMemory() API to SkColorSpace (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Added unit test Created 4 years, 4 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 | « src/core/SkColorSpace.cpp ('k') | no next file » | 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"
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 } else { 159 } else {
160 REPORTER_ASSERT(r, SkColorSpace::k2Dot2Curve_GammaNamed == cs->g ammaNamed()); 160 REPORTER_ASSERT(r, SkColorSpace::k2Dot2Curve_GammaNamed == cs->g ammaNamed());
161 } 161 }
162 } 162 }
163 } 163 }
164 164
165 SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType); 165 SkImageInfo info = SkImageInfo::MakeS32(10, 10, kPremul_SkAlphaType);
166 REPORTER_ASSERT(r, info.gammaCloseToSRGB()); 166 REPORTER_ASSERT(r, info.gammaCloseToSRGB());
167 } 167 }
168 168
169 static void matrix_equals(skiatest::Reporter* r, SkColorSpace* space1, SkColorSp ace* space2) {
170 REPORTER_ASSERT(r, space1->xyz().getFloat(0, 0) == space2->xyz().getFloat(0, 0));
171 REPORTER_ASSERT(r, space1->xyz().getFloat(0, 1) == space2->xyz().getFloat(0, 1));
172 REPORTER_ASSERT(r, space1->xyz().getFloat(0, 2) == space2->xyz().getFloat(0, 2));
173 REPORTER_ASSERT(r, space1->xyz().getFloat(0, 3) == space2->xyz().getFloat(0, 3));
174 REPORTER_ASSERT(r, space1->xyz().getFloat(1, 0) == space2->xyz().getFloat(1, 0));
175 REPORTER_ASSERT(r, space1->xyz().getFloat(1, 1) == space2->xyz().getFloat(1, 1));
176 REPORTER_ASSERT(r, space1->xyz().getFloat(1, 2) == space2->xyz().getFloat(1, 2));
177 REPORTER_ASSERT(r, space1->xyz().getFloat(1, 3) == space2->xyz().getFloat(1, 3));
178 REPORTER_ASSERT(r, space1->xyz().getFloat(2, 0) == space2->xyz().getFloat(2, 0));
179 REPORTER_ASSERT(r, space1->xyz().getFloat(2, 1) == space2->xyz().getFloat(2, 1));
180 REPORTER_ASSERT(r, space1->xyz().getFloat(2, 2) == space2->xyz().getFloat(2, 2));
181 REPORTER_ASSERT(r, space1->xyz().getFloat(2, 3) == space2->xyz().getFloat(2, 3));
182 REPORTER_ASSERT(r, space1->xyz().getFloat(3, 0) == space2->xyz().getFloat(3, 0));
183 REPORTER_ASSERT(r, space1->xyz().getFloat(3, 1) == space2->xyz().getFloat(3, 1));
184 REPORTER_ASSERT(r, space1->xyz().getFloat(3, 2) == space2->xyz().getFloat(3, 2));
185 REPORTER_ASSERT(r, space1->xyz().getFloat(3, 3) == space2->xyz().getFloat(3, 3));
186 }
187
169 static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNa med) { 188 static void test_serialize(skiatest::Reporter* r, SkColorSpace* space, bool isNa med) {
170 sk_sp<SkData> data = space->serialize(); 189 sk_sp<SkData> data1 = space->serialize();
171 sk_sp<SkColorSpace> newSpace = SkColorSpace::Deserialize(data->data(), data- >size()); 190
191 size_t bytes = space->writeToMemory(nullptr);
192 sk_sp<SkData> data2 = SkData::MakeUninitialized(bytes);
193 space->writeToMemory(data2->writable_data());
194
195 sk_sp<SkColorSpace> newSpace1 = SkColorSpace::Deserialize(data1->data(), dat a1->size());
196 sk_sp<SkColorSpace> newSpace2 = SkColorSpace::Deserialize(data2->data(), dat a2->size());
172 197
173 if (isNamed) { 198 if (isNamed) {
174 REPORTER_ASSERT(r, space == newSpace.get()); 199 REPORTER_ASSERT(r, space == newSpace1.get());
200 REPORTER_ASSERT(r, space == newSpace2.get());
175 } else { 201 } else {
176 REPORTER_ASSERT(r, space->gammaNamed() == newSpace->gammaNamed()); 202 REPORTER_ASSERT(r, space->gammaNamed() == newSpace1->gammaNamed());
177 203 REPORTER_ASSERT(r, space->gammaNamed() == newSpace2->gammaNamed());
178 REPORTER_ASSERT(r, space->xyz().getFloat(0, 0) == newSpace->xyz().getFlo at(0, 0)); 204 matrix_equals(r, space, newSpace1.get());
179 REPORTER_ASSERT(r, space->xyz().getFloat(0, 1) == newSpace->xyz().getFlo at(0, 1)); 205 matrix_equals(r, space, newSpace2.get());
180 REPORTER_ASSERT(r, space->xyz().getFloat(0, 2) == newSpace->xyz().getFlo at(0, 2));
181 REPORTER_ASSERT(r, space->xyz().getFloat(0, 3) == newSpace->xyz().getFlo at(0, 3));
182 REPORTER_ASSERT(r, space->xyz().getFloat(1, 0) == newSpace->xyz().getFlo at(1, 0));
183 REPORTER_ASSERT(r, space->xyz().getFloat(1, 1) == newSpace->xyz().getFlo at(1, 1));
184 REPORTER_ASSERT(r, space->xyz().getFloat(1, 2) == newSpace->xyz().getFlo at(1, 2));
185 REPORTER_ASSERT(r, space->xyz().getFloat(1, 3) == newSpace->xyz().getFlo at(1, 3));
186 REPORTER_ASSERT(r, space->xyz().getFloat(2, 0) == newSpace->xyz().getFlo at(2, 0));
187 REPORTER_ASSERT(r, space->xyz().getFloat(2, 1) == newSpace->xyz().getFlo at(2, 1));
188 REPORTER_ASSERT(r, space->xyz().getFloat(2, 2) == newSpace->xyz().getFlo at(2, 2));
189 REPORTER_ASSERT(r, space->xyz().getFloat(2, 3) == newSpace->xyz().getFlo at(2, 3));
190 REPORTER_ASSERT(r, space->xyz().getFloat(3, 0) == newSpace->xyz().getFlo at(3, 0));
191 REPORTER_ASSERT(r, space->xyz().getFloat(3, 1) == newSpace->xyz().getFlo at(3, 1));
192 REPORTER_ASSERT(r, space->xyz().getFloat(3, 2) == newSpace->xyz().getFlo at(3, 2));
193 REPORTER_ASSERT(r, space->xyz().getFloat(3, 3) == newSpace->xyz().getFlo at(3, 3));
194 } 206 }
195 } 207 }
196 208
197 DEF_TEST(ColorSpace_Serialize, r) { 209 DEF_TEST(ColorSpace_Serialize, r) {
198 test_serialize(r, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named).get(), t rue); 210 test_serialize(r, SkColorSpace::NewNamed(SkColorSpace::kSRGB_Named).get(), t rue);
199 test_serialize(r, SkColorSpace::NewNamed(SkColorSpace::kAdobeRGB_Named).get( ), true); 211 test_serialize(r, SkColorSpace::NewNamed(SkColorSpace::kAdobeRGB_Named).get( ), true);
200 212
201 sk_sp<SkData> monitorData = SkData::MakeFromFileName( 213 sk_sp<SkData> monitorData = SkData::MakeFromFileName(
202 GetResourcePath("monitor_profiles/HP_ZR30w.icc").c_str()); 214 GetResourcePath("monitor_profiles/HP_ZR30w.icc").c_str());
203 test_serialize(r, SkColorSpace::NewICC(monitorData->data(), monitorData->siz e()).get(), false); 215 test_serialize(r, SkColorSpace::NewICC(monitorData->data(), monitorData->siz e()).get(), false);
204 } 216 }
205 217
OLDNEW
« no previous file with comments | « src/core/SkColorSpace.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698