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

Side by Side Diff: tests/CodecTest.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 | « src/gpu/GrColorSpaceXform.cpp ('k') | tests/ColorSpaceTest.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 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_XYZ.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(SkColorSpace_Base::Type::kXYZ == as_CSB(infoF16.colorSpace())->type ());
1196 SkColorSpace_XYZ* csXYZ = static_cast<SkColorSpace_XYZ*>(infoF16.colorSpace( ));
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 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
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 }
1365 1367
1366 DEF_TEST(Codec_IcoIntOverflow, r) { 1368 DEF_TEST(Codec_IcoIntOverflow, r) {
1367 // ASAN will complain if there is an issue. 1369 // ASAN will complain if there is an issue.
1368 SkBitmap bitmap; 1370 SkBitmap bitmap;
1369 const bool success = GetResourceAsBitmap("invalid_images/int_overflow.ico", &bitmap); 1371 const bool success = GetResourceAsBitmap("invalid_images/int_overflow.ico", &bitmap);
1370 REPORTER_ASSERT(r, !success); 1372 REPORTER_ASSERT(r, !success);
1371 } 1373 }
OLDNEW
« no previous file with comments | « src/gpu/GrColorSpaceXform.cpp ('k') | tests/ColorSpaceTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698