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

Side by Side Diff: tests/CodecTest.cpp

Issue 2389983002: Refactored SkColorSpace and added in a Lab PCS GM (Closed)
Patch Set: Responding to comments 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 2015 Google Inc. 2 * Copyright 2015 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 "SkAndroidCodec.h" 9 #include "SkAndroidCodec.h"
10 #include "SkBitmap.h" 10 #include "SkBitmap.h"
11 #include "SkCodec.h" 11 #include "SkCodec.h"
12 #include "SkCodecImageGenerator.h" 12 #include "SkCodecImageGenerator.h"
13 #include "SkColorSpace_Base.h" 13 #include "SkColorSpace_XYZTRC.h"
14 #include "SkData.h" 14 #include "SkData.h"
15 #include "SkImageEncoder.h" 15 #include "SkImageEncoder.h"
16 #include "SkFrontBufferedStream.h" 16 #include "SkFrontBufferedStream.h"
17 #include "SkMD5.h" 17 #include "SkMD5.h"
18 #include "SkRandom.h" 18 #include "SkRandom.h"
19 #include "SkStream.h" 19 #include "SkStream.h"
20 #include "SkStreamPriv.h" 20 #include "SkStreamPriv.h"
21 #include "SkPngChunkReader.h" 21 #include "SkPngChunkReader.h"
22 #include "Test.h" 22 #include "Test.h"
23 23
(...skipping 1161 matching lines...) Expand 10 before | Expand all | Expand 10 after
1185 REPORTER_ASSERT(r, SkCodec::kUnimplemented == result); 1185 REPORTER_ASSERT(r, SkCodec::kUnimplemented == result);
1186 } 1186 }
1187 1187
1188 result = codec->startIncrementalDecode(infoF16, bm.getPixels(), bm.rowBytes( )); 1188 result = codec->startIncrementalDecode(infoF16, bm.getPixels(), bm.rowBytes( ));
1189 if (supportsIncrementalDecoder) { 1189 if (supportsIncrementalDecoder) {
1190 REPORTER_ASSERT(r, SkCodec::kInvalidConversion == result); 1190 REPORTER_ASSERT(r, SkCodec::kInvalidConversion == result);
1191 } else { 1191 } else {
1192 REPORTER_ASSERT(r, SkCodec::kUnimplemented == result); 1192 REPORTER_ASSERT(r, SkCodec::kUnimplemented == result);
1193 } 1193 }
1194 1194
1195 infoF16 = infoF16.makeColorSpace(as_CSB(infoF16.colorSpace())->makeLinearGam ma()); 1195 SkASSERT(as_CSB(infoF16.colorSpace())->toXYZD50());
1196 SkColorSpace_XYZTRC* csXYZ = static_cast<SkColorSpace_XYZTRC*>(infoF16.color Space());
1197 infoF16 = infoF16.makeColorSpace(csXYZ->makeLinearGamma());
1196 result = codec->getPixels(infoF16, bm.getPixels(), bm.rowBytes()); 1198 result = codec->getPixels(infoF16, bm.getPixels(), bm.rowBytes());
1197 REPORTER_ASSERT(r, SkCodec::kSuccess == result); 1199 REPORTER_ASSERT(r, SkCodec::kSuccess == result);
1198 result = codec->startScanlineDecode(infoF16); 1200 result = codec->startScanlineDecode(infoF16);
1199 if (supportsScanlineDecoder) { 1201 if (supportsScanlineDecoder) {
1200 REPORTER_ASSERT(r, SkCodec::kSuccess == result); 1202 REPORTER_ASSERT(r, SkCodec::kSuccess == result);
1201 } else { 1203 } else {
1202 REPORTER_ASSERT(r, SkCodec::kUnimplemented == result); 1204 REPORTER_ASSERT(r, SkCodec::kUnimplemented == result);
1203 } 1205 }
1204 1206
1205 result = codec->startIncrementalDecode(infoF16, bm.getPixels(), bm.rowBytes( )); 1207 result = codec->startIncrementalDecode(infoF16, bm.getPixels(), bm.rowBytes( ));
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
1355 auto result = codec->startIncrementalDecode(info, bm.getPixels(), bm.rowByte s()); 1357 auto result = codec->startIncrementalDecode(info, bm.getPixels(), bm.rowByte s());
1356 REPORTER_ASSERT(r, result == SkCodec::kSuccess); 1358 REPORTER_ASSERT(r, result == SkCodec::kSuccess);
1357 1359
1358 // This is an arbitrary value. The important fact is that it is not zero, an d rowsDecoded 1360 // This is an arbitrary value. The important fact is that it is not zero, an d rowsDecoded
1359 // should get set to zero by incrementalDecode. 1361 // should get set to zero by incrementalDecode.
1360 int rowsDecoded = 77; 1362 int rowsDecoded = 77;
1361 result = codec->incrementalDecode(&rowsDecoded); 1363 result = codec->incrementalDecode(&rowsDecoded);
1362 REPORTER_ASSERT(r, result == SkCodec::kIncompleteInput); 1364 REPORTER_ASSERT(r, result == SkCodec::kIncompleteInput);
1363 REPORTER_ASSERT(r, rowsDecoded == 0); 1365 REPORTER_ASSERT(r, rowsDecoded == 0);
1364 } 1366 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698