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

Side by Side Diff: tests/BitmapHasherTest.cpp

Issue 510423005: make allocPixels throw on failure (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: rebase Created 6 years, 3 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/BitmapCopyTest.cpp ('k') | tests/BitmapTest.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 2013 Google Inc. 2 * Copyright 2013 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 "SkBitmapHasher.h" 8 #include "SkBitmapHasher.h"
9 9
10 #include "SkBitmap.h" 10 #include "SkBitmap.h"
11 #include "SkColor.h" 11 #include "SkColor.h"
12 #include "Test.h" 12 #include "Test.h"
13 13
14 // Word size that is large enough to hold results of any checksum type. 14 // Word size that is large enough to hold results of any checksum type.
15 typedef uint64_t checksum_result; 15 typedef uint64_t checksum_result;
16 16
17 // Fill in bitmap with test data. 17 // Fill in bitmap with test data.
18 static void CreateTestBitmap(SkBitmap* bitmap, int width, int height, 18 static void CreateTestBitmap(SkBitmap* bitmap, int width, int height,
19 SkColor color, skiatest::Reporter* reporter) { 19 SkColor color, skiatest::Reporter* reporter) {
20 SkImageInfo info = SkImageInfo::MakeN32(width, height, kOpaque_SkAlphaType); 20 bitmap->allocN32Pixels(width, height, kOpaque_SkAlphaType);
21 REPORTER_ASSERT(reporter, bitmap->allocPixels(info));
22 bitmap->eraseColor(color); 21 bitmap->eraseColor(color);
23 } 22 }
24 23
25 DEF_TEST(BitmapHasher, reporter) { 24 DEF_TEST(BitmapHasher, reporter) {
26 // Test SkBitmapHasher 25 // Test SkBitmapHasher
27 SkBitmap bitmap; 26 SkBitmap bitmap;
28 uint64_t digest; 27 uint64_t digest;
29 // initial test case 28 // initial test case
30 CreateTestBitmap(&bitmap, 333, 555, SK_ColorBLUE, reporter); 29 CreateTestBitmap(&bitmap, 333, 555, SK_ColorBLUE, reporter);
31 REPORTER_ASSERT(reporter, SkBitmapHasher::ComputeDigest(bitmap, &digest)); 30 REPORTER_ASSERT(reporter, SkBitmapHasher::ComputeDigest(bitmap, &digest));
32 REPORTER_ASSERT(reporter, digest == 0xfb2903562766ef87ULL); 31 REPORTER_ASSERT(reporter, digest == 0xfb2903562766ef87ULL);
33 // same pixel data but different dimensions should yield a different checksu m 32 // same pixel data but different dimensions should yield a different checksu m
34 CreateTestBitmap(&bitmap, 555, 333, SK_ColorBLUE, reporter); 33 CreateTestBitmap(&bitmap, 555, 333, SK_ColorBLUE, reporter);
35 REPORTER_ASSERT(reporter, SkBitmapHasher::ComputeDigest(bitmap, &digest)); 34 REPORTER_ASSERT(reporter, SkBitmapHasher::ComputeDigest(bitmap, &digest));
36 REPORTER_ASSERT(reporter, digest == 0xfe04023fb97d0f61ULL); 35 REPORTER_ASSERT(reporter, digest == 0xfe04023fb97d0f61ULL);
37 // same dimensions but different color should yield a different checksum 36 // same dimensions but different color should yield a different checksum
38 CreateTestBitmap(&bitmap, 555, 333, SK_ColorGREEN, reporter); 37 CreateTestBitmap(&bitmap, 555, 333, SK_ColorGREEN, reporter);
39 REPORTER_ASSERT(reporter, SkBitmapHasher::ComputeDigest(bitmap, &digest)); 38 REPORTER_ASSERT(reporter, SkBitmapHasher::ComputeDigest(bitmap, &digest));
40 REPORTER_ASSERT(reporter, digest == 0x2423c51cad6d1edcULL); 39 REPORTER_ASSERT(reporter, digest == 0x2423c51cad6d1edcULL);
41 } 40 }
OLDNEW
« no previous file with comments | « tests/BitmapCopyTest.cpp ('k') | tests/BitmapTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698