blob: 7049c0c29909eef30d0a01d13f3f6802879795f3 [file] [log] [blame]
epoger@google.comec3ed6a2011-07-28 14:26:00 +00001
2/*
3 * Copyright 2011 Google Inc.
4 *
5 * Use of this source code is governed by a BSD-style license that can be
6 * found in the LICENSE file.
7 */
reed@google.com2ade0862011-03-17 17:48:04 +00008#include "Test.h"
9#include "SkBitmap.h"
10#include "SkCanvas.h"
reed@google.comcef5bb42013-10-15 20:05:24 +000011#include "SkData.h"
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000012#include "SkDiscardableMemoryPool.h"
13#include "SkDiscardablePixelRef.h"
reed@google.comcef5bb42013-10-15 20:05:24 +000014#include "SkPaint.h"
reed@google.com99c114e2012-05-03 20:14:26 +000015#include "SkShader.h"
reed@google.comcef5bb42013-10-15 20:05:24 +000016#include "SkSurface.h"
reed@google.comad514302013-01-02 20:19:45 +000017#include "SkRandom.h"
18#include "SkMatrixUtils.h"
19
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000020namespace {
reed@google.comcef5bb42013-10-15 20:05:24 +000021// A BitmapFactory that always fails when asked to return pixels.
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000022class FailureImageGenerator : public SkImageGenerator {
23public:
24 FailureImageGenerator() { }
25 virtual ~FailureImageGenerator() { }
26 virtual bool getInfo(SkImageInfo* info) {
27 info->fWidth = 100;
28 info->fHeight = 100;
29 info->fColorType = kPMColor_SkColorType;
reed@google.comcef5bb42013-10-15 20:05:24 +000030 info->fAlphaType = kPremul_SkAlphaType;
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000031 return true;
reed@google.comcef5bb42013-10-15 20:05:24 +000032 }
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000033 virtual bool getPixels(const SkImageInfo& info,
34 void* pixels,
35 size_t rowBytes) SK_OVERRIDE {
36 // this will deliberately return false if they are asking us
37 // to decode into pixels.
38 return false;
39 }
40};
41} // namespace
reed@google.comcef5bb42013-10-15 20:05:24 +000042
43// crbug.com/295895
44// Crashing in skia when a pixelref fails in lockPixels
45//
46static void test_faulty_pixelref(skiatest::Reporter* reporter) {
reed@google.com3e895242013-12-06 18:41:33 +000047 SkImageInfo info;
48 info.fWidth = 100;
49 info.fHeight = 100;
50 info.fColorType = kPMColor_SkColorType;
51 info.fAlphaType = kPremul_SkAlphaType;
52
reed@google.comcef5bb42013-10-15 20:05:24 +000053 // need a cache, but don't expect to use it, so the budget is not critical
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000054 SkAutoTUnref<SkDiscardableMemoryPool> pool(SkNEW_ARGS(SkDiscardableMemoryPool,
55 (10 * 1000, NULL)));
reed@google.comcef5bb42013-10-15 20:05:24 +000056 SkBitmap bm;
halcanary@google.com2c7c7ee2013-12-05 18:31:42 +000057 bool installSuccess = SkDiscardablePixelRef::Install(SkNEW(FailureImageGenerator), &bm, pool);
58 REPORTER_ASSERT(reporter, installSuccess);
reed@google.com3e895242013-12-06 18:41:33 +000059
reed@google.comcef5bb42013-10-15 20:05:24 +000060 // now our bitmap has a pixelref, but we know it will fail to lock
61
62 SkAutoTUnref<SkSurface> surface(SkSurface::NewRasterPMColor(200, 200));
63 SkCanvas* canvas = surface->getCanvas();
64
65 const SkPaint::FilterLevel levels[] = {
66 SkPaint::kNone_FilterLevel,
67 SkPaint::kLow_FilterLevel,
68 SkPaint::kMedium_FilterLevel,
69 SkPaint::kHigh_FilterLevel,
70 };
71
72 SkPaint paint;
73 canvas->scale(2, 2); // need a scale, otherwise we may ignore filtering
74 for (size_t i = 0; i < SK_ARRAY_COUNT(levels); ++i) {
75 paint.setFilterLevel(levels[i]);
76 canvas->drawBitmap(bm, 0, 0, &paint);
77 }
78}
79
80///////////////////////////////////////////////////////////////////////////////
81
commit-bot@chromium.orge0e7cfe2013-09-09 20:09:12 +000082static void rand_matrix(SkMatrix* mat, SkRandom& rand, unsigned mask) {
reed@google.comad514302013-01-02 20:19:45 +000083 mat->setIdentity();
84 if (mask & SkMatrix::kTranslate_Mask) {
85 mat->postTranslate(rand.nextSScalar1(), rand.nextSScalar1());
86 }
87 if (mask & SkMatrix::kScale_Mask) {
88 mat->postScale(rand.nextSScalar1(), rand.nextSScalar1());
89 }
90 if (mask & SkMatrix::kAffine_Mask) {
91 mat->postRotate(rand.nextSScalar1() * 360);
92 }
93 if (mask & SkMatrix::kPerspective_Mask) {
94 mat->setPerspX(rand.nextSScalar1());
95 mat->setPerspY(rand.nextSScalar1());
96 }
97}
98
commit-bot@chromium.orge0e7cfe2013-09-09 20:09:12 +000099static void rand_size(SkISize* size, SkRandom& rand) {
reed@google.comae573582013-01-03 15:22:40 +0000100 size->set(rand.nextU() & 0xFFFF, rand.nextU() & 0xFFFF);
101}
102
103static bool treat_as_sprite(const SkMatrix& mat, const SkISize& size,
104 unsigned bits) {
105 return SkTreatAsSprite(mat, size.width(), size.height(), bits);
reed@google.comad514302013-01-02 20:19:45 +0000106}
107
108static void test_treatAsSprite(skiatest::Reporter* reporter) {
109 const unsigned bilerBits = kSkSubPixelBitsForBilerp;
110
111 SkMatrix mat;
reed@google.comae573582013-01-03 15:22:40 +0000112 SkISize size;
commit-bot@chromium.orge0e7cfe2013-09-09 20:09:12 +0000113 SkRandom rand;
reed@google.comad514302013-01-02 20:19:45 +0000114
115 // assert: translate-only no-filter can always be treated as sprite
116 for (int i = 0; i < 1000; ++i) {
117 rand_matrix(&mat, rand, SkMatrix::kTranslate_Mask);
118 for (int j = 0; j < 1000; ++j) {
reed@google.comae573582013-01-03 15:22:40 +0000119 rand_size(&size, rand);
120 REPORTER_ASSERT(reporter, treat_as_sprite(mat, size, 0));
reed@google.comad514302013-01-02 20:19:45 +0000121 }
122 }
skia.committer@gmail.com422188f2013-01-03 02:01:32 +0000123
reed@google.comad514302013-01-02 20:19:45 +0000124 // assert: rotate/perspect is never treated as sprite
125 for (int i = 0; i < 1000; ++i) {
126 rand_matrix(&mat, rand, SkMatrix::kAffine_Mask | SkMatrix::kPerspective_Mask);
127 for (int j = 0; j < 1000; ++j) {
reed@google.comae573582013-01-03 15:22:40 +0000128 rand_size(&size, rand);
129 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, 0));
130 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, bilerBits));
reed@google.comad514302013-01-02 20:19:45 +0000131 }
132 }
133
reed@google.comae573582013-01-03 15:22:40 +0000134 size.set(500, 600);
reed@google.comad514302013-01-02 20:19:45 +0000135
commit-bot@chromium.org4b413c82013-11-25 19:44:07 +0000136 const SkScalar tooMuchSubpixel = 100.1f;
reed@google.comad514302013-01-02 20:19:45 +0000137 mat.setTranslate(tooMuchSubpixel, 0);
reed@google.comae573582013-01-03 15:22:40 +0000138 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, bilerBits));
reed@google.comad514302013-01-02 20:19:45 +0000139 mat.setTranslate(0, tooMuchSubpixel);
reed@google.comae573582013-01-03 15:22:40 +0000140 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, bilerBits));
reed@google.comad514302013-01-02 20:19:45 +0000141
commit-bot@chromium.org4b413c82013-11-25 19:44:07 +0000142 const SkScalar tinySubPixel = 100.02f;
reed@google.comad514302013-01-02 20:19:45 +0000143 mat.setTranslate(tinySubPixel, 0);
reed@google.comae573582013-01-03 15:22:40 +0000144 REPORTER_ASSERT(reporter, treat_as_sprite(mat, size, bilerBits));
reed@google.comad514302013-01-02 20:19:45 +0000145 mat.setTranslate(0, tinySubPixel);
reed@google.comae573582013-01-03 15:22:40 +0000146 REPORTER_ASSERT(reporter, treat_as_sprite(mat, size, bilerBits));
skia.committer@gmail.com422188f2013-01-03 02:01:32 +0000147
reed@google.comad514302013-01-02 20:19:45 +0000148 const SkScalar twoThirds = SK_Scalar1 * 2 / 3;
reed@google.comae573582013-01-03 15:22:40 +0000149 const SkScalar bigScale = SkScalarDiv(size.width() + twoThirds, size.width());
reed@google.comad514302013-01-02 20:19:45 +0000150 mat.setScale(bigScale, bigScale);
reed@google.comae573582013-01-03 15:22:40 +0000151 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, false));
152 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, bilerBits));
skia.committer@gmail.com422188f2013-01-03 02:01:32 +0000153
reed@google.comad514302013-01-02 20:19:45 +0000154 const SkScalar oneThird = SK_Scalar1 / 3;
reed@google.comae573582013-01-03 15:22:40 +0000155 const SkScalar smallScale = SkScalarDiv(size.width() + oneThird, size.width());
reed@google.comad514302013-01-02 20:19:45 +0000156 mat.setScale(smallScale, smallScale);
reed@google.comae573582013-01-03 15:22:40 +0000157 REPORTER_ASSERT(reporter, treat_as_sprite(mat, size, false));
158 REPORTER_ASSERT(reporter, !treat_as_sprite(mat, size, bilerBits));
skia.committer@gmail.com422188f2013-01-03 02:01:32 +0000159
reed@google.comad514302013-01-02 20:19:45 +0000160 const SkScalar oneFortyth = SK_Scalar1 / 40;
reed@google.comae573582013-01-03 15:22:40 +0000161 const SkScalar tinyScale = SkScalarDiv(size.width() + oneFortyth, size.width());
reed@google.comad514302013-01-02 20:19:45 +0000162 mat.setScale(tinyScale, tinyScale);
reed@google.comae573582013-01-03 15:22:40 +0000163 REPORTER_ASSERT(reporter, treat_as_sprite(mat, size, false));
164 REPORTER_ASSERT(reporter, treat_as_sprite(mat, size, bilerBits));
reed@google.comad514302013-01-02 20:19:45 +0000165}
reed@google.com99c114e2012-05-03 20:14:26 +0000166
167static void assert_ifDrawnTo(skiatest::Reporter* reporter,
168 const SkBitmap& bm, bool shouldBeDrawn) {
169 for (int y = 0; y < bm.height(); ++y) {
170 for (int x = 0; x < bm.width(); ++x) {
171 if (shouldBeDrawn) {
commit-bot@chromium.orgc2050e32013-07-15 13:10:31 +0000172 if (SK_ColorTRANSPARENT == *bm.getAddr32(x, y)) {
reed@google.com99c114e2012-05-03 20:14:26 +0000173 REPORTER_ASSERT(reporter, false);
174 return;
175 }
176 } else {
177 // should not be drawn
commit-bot@chromium.orgc2050e32013-07-15 13:10:31 +0000178 if (SK_ColorTRANSPARENT != *bm.getAddr32(x, y)) {
reed@google.com99c114e2012-05-03 20:14:26 +0000179 REPORTER_ASSERT(reporter, false);
180 return;
181 }
182 }
183 }
184 }
185}
186
187static void test_wacky_bitmapshader(skiatest::Reporter* reporter,
188 int width, int height, bool shouldBeDrawn) {
189 SkBitmap dev;
190 dev.setConfig(SkBitmap::kARGB_8888_Config, 0x56F, 0x4f6);
191 dev.allocPixels();
junov@google.comdbfac8a2012-12-06 21:47:40 +0000192 dev.eraseColor(SK_ColorTRANSPARENT); // necessary, so we know if we draw to it
rmistry@google.comd6176b02012-08-23 18:14:13 +0000193
reed@google.com99c114e2012-05-03 20:14:26 +0000194 SkMatrix matrix;
rmistry@google.comd6176b02012-08-23 18:14:13 +0000195
reed@google.com99c114e2012-05-03 20:14:26 +0000196 SkCanvas c(dev);
commit-bot@chromium.org4b413c82013-11-25 19:44:07 +0000197 matrix.setAll(-119.34097f,
198 -43.436558f,
199 93489.945f,
200 43.436558f,
201 -119.34097f,
202 123.98426f,
robertphillips@google.com4debcac2012-05-14 16:33:36 +0000203 0, 0, SK_Scalar1);
reed@google.com99c114e2012-05-03 20:14:26 +0000204 c.concat(matrix);
rmistry@google.comd6176b02012-08-23 18:14:13 +0000205
reed@google.com99c114e2012-05-03 20:14:26 +0000206 SkBitmap bm;
207 bm.setConfig(SkBitmap::kARGB_8888_Config, width, height);
208 bm.allocPixels();
209 bm.eraseColor(SK_ColorRED);
rmistry@google.comd6176b02012-08-23 18:14:13 +0000210
reed@google.com99c114e2012-05-03 20:14:26 +0000211 SkShader* s = SkShader::CreateBitmapShader(bm, SkShader::kRepeat_TileMode,
212 SkShader::kRepeat_TileMode);
commit-bot@chromium.org4b413c82013-11-25 19:44:07 +0000213 matrix.setAll(0.0078740157f,
rmistry@google.comd6176b02012-08-23 18:14:13 +0000214 0,
robertphillips@google.com4debcac2012-05-14 16:33:36 +0000215 SkIntToScalar(249),
rmistry@google.comd6176b02012-08-23 18:14:13 +0000216 0,
commit-bot@chromium.org4b413c82013-11-25 19:44:07 +0000217 0.0078740157f,
robertphillips@google.com4debcac2012-05-14 16:33:36 +0000218 SkIntToScalar(239),
219 0, 0, SK_Scalar1);
reed@google.com99c114e2012-05-03 20:14:26 +0000220 s->setLocalMatrix(matrix);
rmistry@google.comd6176b02012-08-23 18:14:13 +0000221
reed@google.com99c114e2012-05-03 20:14:26 +0000222 SkPaint paint;
223 paint.setShader(s)->unref();
rmistry@google.comd6176b02012-08-23 18:14:13 +0000224
reed@google.com99c114e2012-05-03 20:14:26 +0000225 SkRect r = SkRect::MakeXYWH(681, 239, 695, 253);
226 c.drawRect(r, paint);
rmistry@google.comd6176b02012-08-23 18:14:13 +0000227
reed@google.com99c114e2012-05-03 20:14:26 +0000228 assert_ifDrawnTo(reporter, dev, shouldBeDrawn);
229}
230
231/*
232 * Original bug was asserting that the matrix-proc had generated a (Y) value
233 * that was out of range. This led (in the release build) to the sampler-proc
234 * reading memory out-of-bounds of the original bitmap.
235 *
236 * We were numerically overflowing our 16bit coordinates that we communicate
237 * between these two procs. The fixes was in two parts:
238 *
239 * 1. Just don't draw bitmaps larger than 64K-1 in width or height, since we
240 * can't represent those coordinates in our transport format (yet).
241 * 2. Perform an unsigned shift during the calculation, so we don't get
242 * sign-extension bleed when packing the two values (X,Y) into our 32bit
243 * slot.
244 *
245 * This tests exercises the original setup, plus 3 more to ensure that we can,
246 * in fact, handle bitmaps at 64K-1 (assuming we don't exceed the total
247 * memory allocation limit).
248 */
249static void test_giantrepeat_crbug118018(skiatest::Reporter* reporter) {
reed@google.com0da06272012-05-03 20:26:06 +0000250#ifdef SK_SCALAR_IS_FLOAT
reed@google.com99c114e2012-05-03 20:14:26 +0000251 static const struct {
252 int fWidth;
253 int fHeight;
254 bool fExpectedToDraw;
255 } gTests[] = {
256 { 0x1b294, 0x7f, false }, // crbug 118018 (width exceeds 64K)
257 { 0xFFFF, 0x7f, true }, // should draw, test max width
258 { 0x7f, 0xFFFF, true }, // should draw, test max height
259 { 0xFFFF, 0xFFFF, false }, // allocation fails (too much RAM)
260 };
rmistry@google.comd6176b02012-08-23 18:14:13 +0000261
reed@google.com99c114e2012-05-03 20:14:26 +0000262 for (size_t i = 0; i < SK_ARRAY_COUNT(gTests); ++i) {
263 test_wacky_bitmapshader(reporter,
264 gTests[i].fWidth, gTests[i].fHeight,
265 gTests[i].fExpectedToDraw);
266 }
reed@google.com0da06272012-05-03 20:26:06 +0000267#endif
reed@google.com99c114e2012-05-03 20:14:26 +0000268}
reed@google.com0da06272012-05-03 20:26:06 +0000269
reed@google.com99c114e2012-05-03 20:14:26 +0000270///////////////////////////////////////////////////////////////////////////////
reed@google.com2ade0862011-03-17 17:48:04 +0000271
sugoi@google.com54f0d1b2013-02-27 19:17:41 +0000272static void test_nan_antihair() {
reed@google.com6de0bfc2012-03-30 17:43:33 +0000273 SkBitmap bm;
274 bm.setConfig(SkBitmap::kARGB_8888_Config, 20, 20);
275 bm.allocPixels();
276
277 SkCanvas canvas(bm);
278
279 SkPath path;
280 path.moveTo(0, 0);
281 path.lineTo(10, SK_ScalarNaN);
rmistry@google.comd6176b02012-08-23 18:14:13 +0000282
reed@google.com6de0bfc2012-03-30 17:43:33 +0000283 SkPaint paint;
284 paint.setAntiAlias(true);
285 paint.setStyle(SkPaint::kStroke_Style);
rmistry@google.comd6176b02012-08-23 18:14:13 +0000286
reed@google.com6de0bfc2012-03-30 17:43:33 +0000287 // before our fix to SkScan_Antihair.cpp to check for integral NaN (0x800...)
288 // this would trigger an assert/crash.
289 //
290 // see rev. 3558
291 canvas.drawPath(path, paint);
292}
293
reed@google.com2ade0862011-03-17 17:48:04 +0000294static bool check_for_all_zeros(const SkBitmap& bm) {
295 SkAutoLockPixels alp(bm);
296
297 size_t count = bm.width() * bm.bytesPerPixel();
298 for (int y = 0; y < bm.height(); y++) {
299 const uint8_t* ptr = reinterpret_cast<const uint8_t*>(bm.getAddr(0, y));
300 for (size_t i = 0; i < count; i++) {
301 if (ptr[i]) {
302 return false;
303 }
304 }
305 }
306 return true;
307}
308
309static const int gWidth = 256;
310static const int gHeight = 256;
311
312static void create(SkBitmap* bm, SkBitmap::Config config, SkColor color) {
313 bm->setConfig(config, gWidth, gHeight);
314 bm->allocPixels();
315 bm->eraseColor(color);
316}
317
318static void TestDrawBitmapRect(skiatest::Reporter* reporter) {
319 SkBitmap src, dst;
320
321 create(&src, SkBitmap::kARGB_8888_Config, 0xFFFFFFFF);
322 create(&dst, SkBitmap::kARGB_8888_Config, 0);
323
324 SkCanvas canvas(dst);
325
326 SkIRect srcR = { gWidth, 0, gWidth + 16, 16 };
327 SkRect dstR = { 0, 0, SkIntToScalar(16), SkIntToScalar(16) };
328
329 canvas.drawBitmapRect(src, &srcR, dstR, NULL);
330
331 // ensure that we draw nothing if srcR does not intersect the bitmap
332 REPORTER_ASSERT(reporter, check_for_all_zeros(dst));
reed@google.com6de0bfc2012-03-30 17:43:33 +0000333
sugoi@google.com54f0d1b2013-02-27 19:17:41 +0000334 test_nan_antihair();
reed@google.com99c114e2012-05-03 20:14:26 +0000335 test_giantrepeat_crbug118018(reporter);
reed@google.comad514302013-01-02 20:19:45 +0000336
337 test_treatAsSprite(reporter);
reed@google.com23d0ab72013-10-15 20:14:00 +0000338 test_faulty_pixelref(reporter);
reed@google.com2ade0862011-03-17 17:48:04 +0000339}
340
341#include "TestClassDef.h"
342DEFINE_TESTCLASS("DrawBitmapRect", TestDrawBitmapRectClass, TestDrawBitmapRect)