blob: 65427ffd7bbaf28253760a9a40f13e90b1ad869a [file] [log] [blame]
epoger@google.comec3ed6a2011-07-28 14:26:00 +00001/*
2 * Copyright 2011 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +00007
reed@android.com42263962009-05-01 04:00:01 +00008#include "SkBitmap.h"
reed@android.com311c82d2009-05-05 23:13:23 +00009#include "SkRect.h"
tfarina@chromium.org8f6884a2014-01-24 20:56:26 +000010#include "Test.h"
reed@android.com42263962009-05-01 04:00:01 +000011
12static const char* boolStr(bool value) {
13 return value ? "true" : "false";
14}
15
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +000016// these are in the same order as the SkColorType enum
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000017static const char* gColorTypeName[] = {
18 "None", "A8", "565", "4444", "RGBA", "BGRA", "Index8"
reed@android.com42263962009-05-01 04:00:01 +000019};
20
reed@android.comcafc9f92009-08-22 03:44:57 +000021static void report_opaqueness(skiatest::Reporter* reporter, const SkBitmap& src,
22 const SkBitmap& dst) {
halcanary@google.coma9325fa2014-01-10 14:58:10 +000023 ERRORF(reporter, "src %s opaque:%d, dst %s opaque:%d",
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000024 gColorTypeName[src.colorType()], src.isOpaque(),
25 gColorTypeName[dst.colorType()], dst.isOpaque());
reed@android.comcafc9f92009-08-22 03:44:57 +000026}
27
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000028static bool canHaveAlpha(SkColorType ct) {
29 return kRGB_565_SkColorType != ct;
reed@android.comcafc9f92009-08-22 03:44:57 +000030}
31
32// copyTo() should preserve isOpaque when it makes sense
reed@google.com0a6151d2013-10-10 14:44:56 +000033static void test_isOpaque(skiatest::Reporter* reporter,
34 const SkBitmap& srcOpaque, const SkBitmap& srcPremul,
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000035 SkColorType dstColorType) {
reed@android.comcafc9f92009-08-22 03:44:57 +000036 SkBitmap dst;
37
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000038 if (canHaveAlpha(srcPremul.colorType()) && canHaveAlpha(dstColorType)) {
39 REPORTER_ASSERT(reporter, srcPremul.copyTo(&dst, dstColorType));
40 REPORTER_ASSERT(reporter, dst.colorType() == dstColorType);
reed@google.com0a6151d2013-10-10 14:44:56 +000041 if (srcPremul.isOpaque() != dst.isOpaque()) {
42 report_opaqueness(reporter, srcPremul, dst);
reed@android.comcafc9f92009-08-22 03:44:57 +000043 }
44 }
45
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000046 REPORTER_ASSERT(reporter, srcOpaque.copyTo(&dst, dstColorType));
47 REPORTER_ASSERT(reporter, dst.colorType() == dstColorType);
reed@google.com0a6151d2013-10-10 14:44:56 +000048 if (srcOpaque.isOpaque() != dst.isOpaque()) {
49 report_opaqueness(reporter, srcOpaque, dst);
reed@android.comcafc9f92009-08-22 03:44:57 +000050 }
51}
52
reed@google.com0a6151d2013-10-10 14:44:56 +000053static void init_src(const SkBitmap& bitmap) {
weita@google.comf9ab99a2009-05-03 18:23:30 +000054 SkAutoLockPixels lock(bitmap);
reed@android.com42263962009-05-01 04:00:01 +000055 if (bitmap.getPixels()) {
reed@google.com0a6151d2013-10-10 14:44:56 +000056 if (bitmap.getColorTable()) {
reed@google.com9ce6e752011-01-10 14:04:07 +000057 sk_bzero(bitmap.getPixels(), bitmap.getSize());
58 } else {
59 bitmap.eraseColor(SK_ColorWHITE);
60 }
reed@android.com42263962009-05-01 04:00:01 +000061 }
62}
63
reedc5e15a12014-09-29 12:10:27 -070064static SkColorTable* init_ctable() {
reed@android.com42263962009-05-01 04:00:01 +000065 static const SkColor colors[] = {
66 SK_ColorBLACK, SK_ColorRED, SK_ColorGREEN, SK_ColorBLUE, SK_ColorWHITE
67 };
reedc5e15a12014-09-29 12:10:27 -070068 return new SkColorTable(colors, SK_ARRAY_COUNT(colors));
reed@android.com42263962009-05-01 04:00:01 +000069}
70
71struct Pair {
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +000072 SkColorType fColorType;
73 const char* fValid;
reed@android.com42263962009-05-01 04:00:01 +000074};
75
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +000076// Utility functions for copyPixelsTo()/copyPixelsFrom() tests.
77// getPixel()
78// setPixel()
79// getSkConfigName()
80// struct Coordinates
81// reportCopyVerification()
82// writeCoordPixels()
83
84// Utility function to read the value of a given pixel in bm. All
85// values converted to uint32_t for simplification of comparisons.
caryclark@google.com42639cd2012-06-06 12:03:39 +000086static uint32_t getPixel(int x, int y, const SkBitmap& bm) {
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +000087 uint32_t val = 0;
88 uint16_t val16;
rmistry@google.comd6bab022013-12-02 13:50:38 +000089 uint8_t val8;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +000090 SkAutoLockPixels lock(bm);
91 const void* rawAddr = bm.getAddr(x,y);
92
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +000093 switch (bm.bytesPerPixel()) {
94 case 4:
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +000095 memcpy(&val, rawAddr, sizeof(uint32_t));
96 break;
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +000097 case 2:
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +000098 memcpy(&val16, rawAddr, sizeof(uint16_t));
99 val = val16;
100 break;
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000101 case 1:
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000102 memcpy(&val8, rawAddr, sizeof(uint8_t));
103 val = val8;
104 break;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000105 default:
106 break;
107 }
108 return val;
109}
110
111// Utility function to set value of any pixel in bm.
112// bm.getConfig() specifies what format 'val' must be
113// converted to, but at present uint32_t can handle all formats.
caryclark@google.com42639cd2012-06-06 12:03:39 +0000114static void setPixel(int x, int y, uint32_t val, SkBitmap& bm) {
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000115 uint16_t val16;
rmistry@google.comd6bab022013-12-02 13:50:38 +0000116 uint8_t val8;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000117 SkAutoLockPixels lock(bm);
118 void* rawAddr = bm.getAddr(x,y);
119
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000120 switch (bm.bytesPerPixel()) {
121 case 4:
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000122 memcpy(rawAddr, &val, sizeof(uint32_t));
123 break;
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000124 case 2:
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000125 val16 = val & 0xFFFF;
126 memcpy(rawAddr, &val16, sizeof(uint16_t));
127 break;
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000128 case 1:
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000129 val8 = val & 0xFF;
130 memcpy(rawAddr, &val8, sizeof(uint8_t));
131 break;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000132 default:
133 // Ignore.
134 break;
135 }
136}
137
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000138// Helper struct to contain pixel locations, while avoiding need for STL.
139struct Coordinates {
140
141 const int length;
142 SkIPoint* const data;
143
144 explicit Coordinates(int _length): length(_length)
145 , data(new SkIPoint[length]) { }
146
147 ~Coordinates(){
148 delete [] data;
149 }
150
151 SkIPoint* operator[](int i) const {
152 // Use with care, no bounds checking.
153 return data + i;
154 }
155};
156
157// A function to verify that two bitmaps contain the same pixel values
158// at all coordinates indicated by coords. Simplifies verification of
159// copied bitmaps.
caryclark@google.com42639cd2012-06-06 12:03:39 +0000160static void reportCopyVerification(const SkBitmap& bm1, const SkBitmap& bm2,
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000161 Coordinates& coords,
162 const char* msg,
163 skiatest::Reporter* reporter){
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000164 // Confirm all pixels in the list match.
scroggo@google.comd5764e82012-08-22 15:00:05 +0000165 for (int i = 0; i < coords.length; ++i) {
reed92fc2ae2015-05-22 08:06:21 -0700166 uint32_t p1 = getPixel(coords[i]->fX, coords[i]->fY, bm1);
167 uint32_t p2 = getPixel(coords[i]->fX, coords[i]->fY, bm2);
168// SkDebugf("[%d] (%d %d) p1=%x p2=%x\n", i, coords[i]->fX, coords[i]->fY, p1, p2);
169 if (p1 != p2) {
170 ERRORF(reporter, "%s [colortype = %s]", msg, gColorTypeName[bm1.colorType()]);
171 break;
172 }
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000173 }
174}
175
176// Writes unique pixel values at locations specified by coords.
caryclark@google.com42639cd2012-06-06 12:03:39 +0000177static void writeCoordPixels(SkBitmap& bm, const Coordinates& coords) {
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000178 for (int i = 0; i < coords.length; ++i)
179 setPixel(coords[i]->fX, coords[i]->fY, i, bm);
180}
181
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000182static const Pair gPairs[] = {
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000183 { kUnknown_SkColorType, "000000" },
184 { kAlpha_8_SkColorType, "010101" },
reedb184f7f2014-07-13 04:32:32 -0700185 { kIndex_8_SkColorType, "011111" },
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000186 { kRGB_565_SkColorType, "010101" },
187 { kARGB_4444_SkColorType, "010111" },
commit-bot@chromium.org28fcae22014-04-11 17:15:40 +0000188 { kN32_SkColorType, "010111" },
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000189};
commit-bot@chromium.org5c6f1d42014-01-10 18:28:23 +0000190
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000191static const int W = 20;
192static const int H = 33;
193
194static void setup_src_bitmaps(SkBitmap* srcOpaque, SkBitmap* srcPremul,
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000195 SkColorType ct) {
halcanary96fcdcc2015-08-27 07:41:13 -0700196 SkColorTable* ctable = nullptr;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000197 if (kIndex_8_SkColorType == ct) {
reedc5e15a12014-09-29 12:10:27 -0700198 ctable = init_ctable();
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000199 }
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000200
201 srcOpaque->allocPixels(SkImageInfo::Make(W, H, ct, kOpaque_SkAlphaType),
halcanary96fcdcc2015-08-27 07:41:13 -0700202 nullptr, ctable);
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000203 srcPremul->allocPixels(SkImageInfo::Make(W, H, ct, kPremul_SkAlphaType),
halcanary96fcdcc2015-08-27 07:41:13 -0700204 nullptr, ctable);
reedc5e15a12014-09-29 12:10:27 -0700205 SkSafeUnref(ctable);
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000206 init_src(*srcOpaque);
207 init_src(*srcPremul);
208}
209
210DEF_TEST(BitmapCopy_extractSubset, reporter) {
211 for (size_t i = 0; i < SK_ARRAY_COUNT(gPairs); i++) {
212 SkBitmap srcOpaque, srcPremul;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000213 setup_src_bitmaps(&srcOpaque, &srcPremul, gPairs[i].fColorType);
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000214
215 SkBitmap bitmap(srcOpaque);
216 SkBitmap subset;
217 SkIRect r;
218 // Extract a subset which has the same width as the original. This
219 // catches a bug where we cloned the genID incorrectly.
220 r.set(0, 1, W, 3);
221 bitmap.setIsVolatile(true);
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000222 // Relies on old behavior of extractSubset failing if colortype is unknown
223 if (kUnknown_SkColorType != bitmap.colorType() && bitmap.extractSubset(&subset, r)) {
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000224 REPORTER_ASSERT(reporter, subset.width() == W);
225 REPORTER_ASSERT(reporter, subset.height() == 2);
226 REPORTER_ASSERT(reporter, subset.alphaType() == bitmap.alphaType());
227 REPORTER_ASSERT(reporter, subset.isVolatile() == true);
228
229 // Test copying an extracted subset.
230 for (size_t j = 0; j < SK_ARRAY_COUNT(gPairs); j++) {
231 SkBitmap copy;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000232 bool success = subset.copyTo(&copy, gPairs[j].fColorType);
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000233 if (!success) {
234 // Skip checking that success matches fValid, which is redundant
235 // with the code below.
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000236 REPORTER_ASSERT(reporter, gPairs[i].fColorType != gPairs[j].fColorType);
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000237 continue;
238 }
239
240 // When performing a copy of an extracted subset, the gen id should
241 // change.
242 REPORTER_ASSERT(reporter, copy.getGenerationID() != subset.getGenerationID());
243
244 REPORTER_ASSERT(reporter, copy.width() == W);
245 REPORTER_ASSERT(reporter, copy.height() == 2);
246
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000247 if (gPairs[i].fColorType == gPairs[j].fColorType) {
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000248 SkAutoLockPixels alp0(subset);
249 SkAutoLockPixels alp1(copy);
250 // they should both have, or both not-have, a colortable
halcanary96fcdcc2015-08-27 07:41:13 -0700251 bool hasCT = subset.getColorTable() != nullptr;
252 REPORTER_ASSERT(reporter, (copy.getColorTable() != nullptr) == hasCT);
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000253 }
254 }
255 }
256
257 bitmap = srcPremul;
258 bitmap.setIsVolatile(false);
259 if (bitmap.extractSubset(&subset, r)) {
260 REPORTER_ASSERT(reporter, subset.alphaType() == bitmap.alphaType());
261 REPORTER_ASSERT(reporter, subset.isVolatile() == false);
262 }
263 }
264}
265
266DEF_TEST(BitmapCopy, reporter) {
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000267 static const bool isExtracted[] = {
268 false, true
269 };
270
commit-bot@chromium.org5c6f1d42014-01-10 18:28:23 +0000271 for (size_t i = 0; i < SK_ARRAY_COUNT(gPairs); i++) {
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000272 SkBitmap srcOpaque, srcPremul;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000273 setup_src_bitmaps(&srcOpaque, &srcPremul, gPairs[i].fColorType);
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000274
reed@android.com42263962009-05-01 04:00:01 +0000275 for (size_t j = 0; j < SK_ARRAY_COUNT(gPairs); j++) {
scroggo@google.com5ccae2c2014-01-15 16:56:52 +0000276 SkBitmap dst;
reed@android.com42263962009-05-01 04:00:01 +0000277
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000278 bool success = srcPremul.copyTo(&dst, gPairs[j].fColorType);
reed@android.com42263962009-05-01 04:00:01 +0000279 bool expected = gPairs[i].fValid[j] != '0';
280 if (success != expected) {
halcanary@google.coma9325fa2014-01-10 14:58:10 +0000281 ERRORF(reporter, "SkBitmap::copyTo from %s to %s. expected %s "
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000282 "returned %s", gColorTypeName[i], gColorTypeName[j],
halcanary@google.coma9325fa2014-01-10 14:58:10 +0000283 boolStr(expected), boolStr(success));
reed@android.com42263962009-05-01 04:00:01 +0000284 }
reed@google.com1fcd51e2011-01-05 15:50:27 +0000285
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000286 bool canSucceed = srcPremul.canCopyTo(gPairs[j].fColorType);
reed@android.comfbaa88d2009-05-06 17:44:34 +0000287 if (success != canSucceed) {
halcanary@google.coma9325fa2014-01-10 14:58:10 +0000288 ERRORF(reporter, "SkBitmap::copyTo from %s to %s. returned %s "
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000289 "canCopyTo %s", gColorTypeName[i], gColorTypeName[j],
halcanary@google.coma9325fa2014-01-10 14:58:10 +0000290 boolStr(success), boolStr(canSucceed));
reed@android.comfbaa88d2009-05-06 17:44:34 +0000291 }
weita@google.comf9ab99a2009-05-03 18:23:30 +0000292
reed@android.com42263962009-05-01 04:00:01 +0000293 if (success) {
reed@google.com0a6151d2013-10-10 14:44:56 +0000294 REPORTER_ASSERT(reporter, srcPremul.width() == dst.width());
295 REPORTER_ASSERT(reporter, srcPremul.height() == dst.height());
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000296 REPORTER_ASSERT(reporter, dst.colorType() == gPairs[j].fColorType);
297 test_isOpaque(reporter, srcOpaque, srcPremul, dst.colorType());
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000298 if (srcPremul.colorType() == dst.colorType()) {
reed@google.com0a6151d2013-10-10 14:44:56 +0000299 SkAutoLockPixels srcLock(srcPremul);
reed@android.com42263962009-05-01 04:00:01 +0000300 SkAutoLockPixels dstLock(dst);
reed@google.com0a6151d2013-10-10 14:44:56 +0000301 REPORTER_ASSERT(reporter, srcPremul.readyToDraw());
reed@android.com42263962009-05-01 04:00:01 +0000302 REPORTER_ASSERT(reporter, dst.readyToDraw());
reed@google.com0a6151d2013-10-10 14:44:56 +0000303 const char* srcP = (const char*)srcPremul.getAddr(0, 0);
reed@android.com42263962009-05-01 04:00:01 +0000304 const char* dstP = (const char*)dst.getAddr(0, 0);
305 REPORTER_ASSERT(reporter, srcP != dstP);
306 REPORTER_ASSERT(reporter, !memcmp(srcP, dstP,
reed@google.com0a6151d2013-10-10 14:44:56 +0000307 srcPremul.getSize()));
308 REPORTER_ASSERT(reporter, srcPremul.getGenerationID() == dst.getGenerationID());
scroggo@google.comd5764e82012-08-22 15:00:05 +0000309 } else {
reed@google.com0a6151d2013-10-10 14:44:56 +0000310 REPORTER_ASSERT(reporter, srcPremul.getGenerationID() != dst.getGenerationID());
reed@android.com42263962009-05-01 04:00:01 +0000311 }
reed@android.com42263962009-05-01 04:00:01 +0000312 } else {
313 // dst should be unchanged from its initial state
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000314 REPORTER_ASSERT(reporter, dst.colorType() == kUnknown_SkColorType);
reed@android.com42263962009-05-01 04:00:01 +0000315 REPORTER_ASSERT(reporter, dst.width() == 0);
316 REPORTER_ASSERT(reporter, dst.height() == 0);
317 }
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000318 } // for (size_t j = ...
319
320 // Tests for getSafeSize(), getSafeSize64(), copyPixelsTo(),
321 // copyPixelsFrom().
322 //
323 for (size_t copyCase = 0; copyCase < SK_ARRAY_COUNT(isExtracted);
324 ++copyCase) {
325 // Test copying to/from external buffer.
326 // Note: the tests below have hard-coded values ---
327 // Please take care if modifying.
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000328
reed@google.com2cb14802013-06-26 14:35:02 +0000329 // Tests for getSafeSize64().
330 // Test with a very large configuration without pixel buffer
331 // attached.
332 SkBitmap tstSafeSize;
commit-bot@chromium.orga3264e52014-05-30 13:26:10 +0000333 tstSafeSize.setInfo(SkImageInfo::Make(100000000U, 100000000U,
334 gPairs[i].fColorType, kPremul_SkAlphaType));
reed@google.com57212f92013-12-30 14:40:38 +0000335 int64_t safeSize = tstSafeSize.computeSafeSize64();
336 if (safeSize < 0) {
halcanary@google.coma9325fa2014-01-10 14:58:10 +0000337 ERRORF(reporter, "getSafeSize64() negative: %s",
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000338 gColorTypeName[tstSafeSize.colorType()]);
reed@google.com2cb14802013-06-26 14:35:02 +0000339 }
340 bool sizeFail = false;
341 // Compare against hand-computed values.
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000342 switch (gPairs[i].fColorType) {
343 case kUnknown_SkColorType:
reed@google.com2cb14802013-06-26 14:35:02 +0000344 break;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000345
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000346 case kAlpha_8_SkColorType:
347 case kIndex_8_SkColorType:
reed@google.com57212f92013-12-30 14:40:38 +0000348 if (safeSize != 0x2386F26FC10000LL) {
reed@google.com2cb14802013-06-26 14:35:02 +0000349 sizeFail = true;
reed@google.com01c41a52013-12-20 14:24:21 +0000350 }
reed@google.com2cb14802013-06-26 14:35:02 +0000351 break;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000352
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000353 case kRGB_565_SkColorType:
354 case kARGB_4444_SkColorType:
reed@google.com57212f92013-12-30 14:40:38 +0000355 if (safeSize != 0x470DE4DF820000LL) {
reed@google.com2cb14802013-06-26 14:35:02 +0000356 sizeFail = true;
reed@google.com01c41a52013-12-20 14:24:21 +0000357 }
reed@google.com2cb14802013-06-26 14:35:02 +0000358 break;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000359
commit-bot@chromium.org28fcae22014-04-11 17:15:40 +0000360 case kN32_SkColorType:
reed@google.com57212f92013-12-30 14:40:38 +0000361 if (safeSize != 0x8E1BC9BF040000LL) {
reed@google.com2cb14802013-06-26 14:35:02 +0000362 sizeFail = true;
reed@google.com01c41a52013-12-20 14:24:21 +0000363 }
reed@google.com2cb14802013-06-26 14:35:02 +0000364 break;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000365
reed@google.com2cb14802013-06-26 14:35:02 +0000366 default:
367 break;
368 }
369 if (sizeFail) {
halcanary@google.coma9325fa2014-01-10 14:58:10 +0000370 ERRORF(reporter, "computeSafeSize64() wrong size: %s",
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000371 gColorTypeName[tstSafeSize.colorType()]);
reed@google.com2cb14802013-06-26 14:35:02 +0000372 }
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000373
rmistry@google.comd6bab022013-12-02 13:50:38 +0000374 int subW = 2;
375 int subH = 2;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000376
reed@google.com2cb14802013-06-26 14:35:02 +0000377 // Create bitmap to act as source for copies and subsets.
378 SkBitmap src, subset;
halcanary96fcdcc2015-08-27 07:41:13 -0700379 SkColorTable* ct = nullptr;
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000380 if (kIndex_8_SkColorType == src.colorType()) {
reedc5e15a12014-09-29 12:10:27 -0700381 ct = init_ctable();
reed@google.com2cb14802013-06-26 14:35:02 +0000382 }
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000383
reed84825042014-09-02 12:50:45 -0700384 int localSubW;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000385 if (isExtracted[copyCase]) { // A larger image to extract from.
reed84825042014-09-02 12:50:45 -0700386 localSubW = 2 * subW + 1;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000387 } else { // Tests expect a 2x2 bitmap, so make smaller.
reed84825042014-09-02 12:50:45 -0700388 localSubW = subW;
389 }
390 // could fail if we pass kIndex_8 for the colortype
391 if (src.tryAllocPixels(SkImageInfo::Make(localSubW, subH, gPairs[i].fColorType,
392 kPremul_SkAlphaType))) {
393 // failure is fine, as we will notice later on
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000394 }
reed@google.com2cb14802013-06-26 14:35:02 +0000395 SkSafeUnref(ct);
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000396
reed@google.com2cb14802013-06-26 14:35:02 +0000397 // Either copy src or extract into 'subset', which is used
398 // for subsequent calls to copyPixelsTo/From.
399 bool srcReady = false;
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000400 // Test relies on older behavior that extractSubset will fail on
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000401 // kUnknown_SkColorType
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000402 if (kUnknown_SkColorType != src.colorType() &&
403 isExtracted[copyCase]) {
reed@google.com2cb14802013-06-26 14:35:02 +0000404 // The extractedSubset() test case allows us to test copy-
405 // ing when src and dst mave possibly different strides.
406 SkIRect r;
rmistry@google.comd6bab022013-12-02 13:50:38 +0000407 r.set(1, 0, 1 + subW, subH); // 2x2 extracted bitmap
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000408
reed@google.com2cb14802013-06-26 14:35:02 +0000409 srcReady = src.extractSubset(&subset, r);
410 } else {
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000411 srcReady = src.copyTo(&subset);
reed@google.com2cb14802013-06-26 14:35:02 +0000412 }
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000413
reed@google.com2cb14802013-06-26 14:35:02 +0000414 // Not all configurations will generate a valid 'subset'.
415 if (srcReady) {
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000416
reed@google.com2cb14802013-06-26 14:35:02 +0000417 // Allocate our target buffer 'buf' for all copies.
418 // To simplify verifying correctness of copies attach
419 // buf to a SkBitmap, but copies are done using the
420 // raw buffer pointer.
robertphillips@google.come9cd27d2013-10-16 17:48:11 +0000421 const size_t bufSize = subH *
commit-bot@chromium.org8ef51b92014-03-05 13:43:15 +0000422 SkColorTypeMinRowBytes(src.colorType(), subW) * 2;
reed@google.com2cb14802013-06-26 14:35:02 +0000423 SkAutoMalloc autoBuf (bufSize);
424 uint8_t* buf = static_cast<uint8_t*>(autoBuf.get());
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000425
reed@google.com2cb14802013-06-26 14:35:02 +0000426 SkBitmap bufBm; // Attach buf to this bitmap.
427 bool successExpected;
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000428
reed@google.com2cb14802013-06-26 14:35:02 +0000429 // Set up values for each pixel being copied.
430 Coordinates coords(subW * subH);
robertphillips@google.come9cd27d2013-10-16 17:48:11 +0000431 for (int x = 0; x < subW; ++x)
432 for (int y = 0; y < subH; ++y)
reed@google.com2cb14802013-06-26 14:35:02 +0000433 {
434 int index = y * subW + x;
435 SkASSERT(index < coords.length);
436 coords[index]->fX = x;
437 coords[index]->fY = y;
438 }
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000439
reed@google.com2cb14802013-06-26 14:35:02 +0000440 writeCoordPixels(subset, coords);
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000441
reed@google.com2cb14802013-06-26 14:35:02 +0000442 // Test #1 ////////////////////////////////////////////
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000443
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000444 const SkImageInfo info = SkImageInfo::Make(subW, subH,
445 gPairs[i].fColorType,
446 kPremul_SkAlphaType);
reed@google.com2cb14802013-06-26 14:35:02 +0000447 // Before/after comparisons easier if we attach buf
448 // to an appropriately configured SkBitmap.
449 memset(buf, 0xFF, bufSize);
450 // Config with stride greater than src but that fits in buf.
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000451 bufBm.installPixels(info, buf, info.minRowBytes() * 2);
reed@google.com2cb14802013-06-26 14:35:02 +0000452 successExpected = false;
453 // Then attempt to copy with a stride that is too large
454 // to fit in the buffer.
455 REPORTER_ASSERT(reporter,
456 subset.copyPixelsTo(buf, bufSize, bufBm.rowBytes() * 3)
457 == successExpected);
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000458
reed@google.com2cb14802013-06-26 14:35:02 +0000459 if (successExpected)
460 reportCopyVerification(subset, bufBm, coords,
461 "copyPixelsTo(buf, bufSize, 1.5*maxRowBytes)",
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000462 reporter);
463
reed@google.com2cb14802013-06-26 14:35:02 +0000464 // Test #2 ////////////////////////////////////////////
465 // This test should always succeed, but in the case
466 // of extracted bitmaps only because we handle the
467 // issue of getSafeSize(). Without getSafeSize()
468 // buffer overrun/read would occur.
469 memset(buf, 0xFF, bufSize);
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000470 bufBm.installPixels(info, buf, subset.rowBytes());
reed@google.com2cb14802013-06-26 14:35:02 +0000471 successExpected = subset.getSafeSize() <= bufSize;
472 REPORTER_ASSERT(reporter,
473 subset.copyPixelsTo(buf, bufSize) ==
474 successExpected);
475 if (successExpected)
476 reportCopyVerification(subset, bufBm, coords,
477 "copyPixelsTo(buf, bufSize)", reporter);
478
479 // Test #3 ////////////////////////////////////////////
480 // Copy with different stride between src and dst.
481 memset(buf, 0xFF, bufSize);
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000482 bufBm.installPixels(info, buf, subset.rowBytes()+1);
reed@google.com2cb14802013-06-26 14:35:02 +0000483 successExpected = true; // Should always work.
484 REPORTER_ASSERT(reporter,
485 subset.copyPixelsTo(buf, bufSize,
486 subset.rowBytes()+1) == successExpected);
487 if (successExpected)
488 reportCopyVerification(subset, bufBm, coords,
489 "copyPixelsTo(buf, bufSize, rowBytes+1)", reporter);
490
491 // Test #4 ////////////////////////////////////////////
492 // Test copy with stride too small.
493 memset(buf, 0xFF, bufSize);
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000494 bufBm.installPixels(info, buf, info.minRowBytes());
reed@google.com2cb14802013-06-26 14:35:02 +0000495 successExpected = false;
496 // Request copy with stride too small.
497 REPORTER_ASSERT(reporter,
498 subset.copyPixelsTo(buf, bufSize, bufBm.rowBytes()-1)
499 == successExpected);
500 if (successExpected)
501 reportCopyVerification(subset, bufBm, coords,
502 "copyPixelsTo(buf, bufSize, rowBytes()-1)", reporter);
503
504#if 0 // copyPixelsFrom is gone
505 // Test #5 ////////////////////////////////////////////
506 // Tests the case where the source stride is too small
507 // for the source configuration.
508 memset(buf, 0xFF, bufSize);
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000509 bufBm.installPixels(info, buf, info.minRowBytes());
reed@google.com2cb14802013-06-26 14:35:02 +0000510 writeCoordPixels(bufBm, coords);
511 REPORTER_ASSERT(reporter,
512 subset.copyPixelsFrom(buf, bufSize, 1) == false);
513
514 // Test #6 ///////////////////////////////////////////
515 // Tests basic copy from an external buffer to the bitmap.
516 // If the bitmap is "extracted", this also tests the case
517 // where the source stride is different from the dest.
518 // stride.
519 // We've made the buffer large enough to always succeed.
commit-bot@chromium.org8a2ad3c2014-02-23 03:59:35 +0000520 bufBm.installPixels(info, buf, info.minRowBytes());
reed@google.com2cb14802013-06-26 14:35:02 +0000521 writeCoordPixels(bufBm, coords);
522 REPORTER_ASSERT(reporter,
523 subset.copyPixelsFrom(buf, bufSize, bufBm.rowBytes()) ==
524 true);
525 reportCopyVerification(bufBm, subset, coords,
526 "copyPixelsFrom(buf, bufSize)",
527 reporter);
528
529 // Test #7 ////////////////////////////////////////////
530 // Tests the case where the source buffer is too small
531 // for the transfer.
532 REPORTER_ASSERT(reporter,
533 subset.copyPixelsFrom(buf, 1, subset.rowBytes()) ==
534 false);
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000535
reed@google.comab77aaf2011-11-01 16:03:35 +0000536#endif
wjmaclean@chromium.org86bff1f2010-11-16 20:22:41 +0000537 }
538 } // for (size_t copyCase ...
reed@android.com42263962009-05-01 04:00:01 +0000539 }
540}
reedb184f7f2014-07-13 04:32:32 -0700541
542#include "SkColorPriv.h"
543#include "SkUtils.h"
544
545/**
546 * Construct 4x4 pixels where we can look at a color and determine where it should be in the grid.
547 * alpha = 0xFF, blue = 0x80, red = x, green = y
548 */
549static void fill_4x4_pixels(SkPMColor colors[16]) {
550 for (int y = 0; y < 4; ++y) {
551 for (int x = 0; x < 4; ++x) {
552 colors[y*4+x] = SkPackARGB32(0xFF, x, y, 0x80);
553 }
554 }
555}
556
557static bool check_4x4_pixel(SkPMColor color, unsigned x, unsigned y) {
558 SkASSERT(x < 4 && y < 4);
559 return 0xFF == SkGetPackedA32(color) &&
560 x == SkGetPackedR32(color) &&
561 y == SkGetPackedG32(color) &&
562 0x80 == SkGetPackedB32(color);
563}
564
565/**
566 * Fill with all zeros, which will never match any value from fill_4x4_pixels
567 */
568static void clear_4x4_pixels(SkPMColor colors[16]) {
569 sk_memset32(colors, 0, 16);
570}
571
572// Much of readPixels is exercised by copyTo testing, since readPixels is the backend for that
573// method. Here we explicitly test subset copies.
574//
575DEF_TEST(BitmapReadPixels, reporter) {
576 const int W = 4;
577 const int H = 4;
578 const size_t rowBytes = W * sizeof(SkPMColor);
579 const SkImageInfo srcInfo = SkImageInfo::MakeN32Premul(W, H);
580 SkPMColor srcPixels[16];
581 fill_4x4_pixels(srcPixels);
582 SkBitmap srcBM;
583 srcBM.installPixels(srcInfo, srcPixels, rowBytes);
584
585 SkImageInfo dstInfo = SkImageInfo::MakeN32Premul(W, H);
586 SkPMColor dstPixels[16];
587
588 const struct {
589 bool fExpectedSuccess;
590 SkIPoint fRequestedSrcLoc;
591 SkISize fRequestedDstSize;
592 // If fExpectedSuccess, check these, otherwise ignore
593 SkIPoint fExpectedDstLoc;
594 SkIRect fExpectedSrcR;
595 } gRec[] = {
596 { true, { 0, 0 }, { 4, 4 }, { 0, 0 }, { 0, 0, 4, 4 } },
597 { true, { 1, 1 }, { 2, 2 }, { 0, 0 }, { 1, 1, 3, 3 } },
598 { true, { 2, 2 }, { 4, 4 }, { 0, 0 }, { 2, 2, 4, 4 } },
599 { true, {-1,-1 }, { 2, 2 }, { 1, 1 }, { 0, 0, 1, 1 } },
600 { false, {-1,-1 }, { 1, 1 }, { 0, 0 }, { 0, 0, 0, 0 } },
601 };
602
603 for (size_t i = 0; i < SK_ARRAY_COUNT(gRec); ++i) {
604 clear_4x4_pixels(dstPixels);
605
reede5ea5002014-09-03 11:54:58 -0700606 dstInfo = dstInfo.makeWH(gRec[i].fRequestedDstSize.width(),
607 gRec[i].fRequestedDstSize.height());
reedb184f7f2014-07-13 04:32:32 -0700608 bool success = srcBM.readPixels(dstInfo, dstPixels, rowBytes,
609 gRec[i].fRequestedSrcLoc.x(), gRec[i].fRequestedSrcLoc.y());
610
611 REPORTER_ASSERT(reporter, gRec[i].fExpectedSuccess == success);
612 if (success) {
613 const SkIRect srcR = gRec[i].fExpectedSrcR;
614 const int dstX = gRec[i].fExpectedDstLoc.x();
615 const int dstY = gRec[i].fExpectedDstLoc.y();
616 // Walk the dst pixels, and check if we got what we expected
617 for (int y = 0; y < H; ++y) {
618 for (int x = 0; x < W; ++x) {
619 SkPMColor dstC = dstPixels[y*4+x];
620 // get into src coordinates
621 int sx = x - dstX + srcR.x();
622 int sy = y - dstY + srcR.y();
623 if (srcR.contains(sx, sy)) {
624 REPORTER_ASSERT(reporter, check_4x4_pixel(dstC, sx, sy));
625 } else {
626 REPORTER_ASSERT(reporter, 0 == dstC);
627 }
628 }
629 }
630 }
631 }
632}
633