epoger@google.com | ec3ed6a | 2011-07-28 14:26:00 +0000 | [diff] [blame] | 1 | |
| 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.com | 2ade086 | 2011-03-17 17:48:04 +0000 | [diff] [blame] | 8 | #include "Test.h" |
| 9 | #include "SkBitmap.h" |
| 10 | #include "SkCanvas.h" |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 11 | #include "SkShader.h" |
| 12 | |
reed@google.com | 0da0627 | 2012-05-03 20:26:06 +0000 | [diff] [blame] | 13 | #ifdef SK_SCALAR_IS_FLOAT |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 14 | static void assert_ifDrawnTo(skiatest::Reporter* reporter, |
| 15 | const SkBitmap& bm, bool shouldBeDrawn) { |
| 16 | for (int y = 0; y < bm.height(); ++y) { |
| 17 | for (int x = 0; x < bm.width(); ++x) { |
| 18 | if (shouldBeDrawn) { |
| 19 | if (0 == *bm.getAddr32(x, y)) { |
| 20 | REPORTER_ASSERT(reporter, false); |
| 21 | return; |
| 22 | } |
| 23 | } else { |
| 24 | // should not be drawn |
| 25 | if (*bm.getAddr32(x, y)) { |
| 26 | REPORTER_ASSERT(reporter, false); |
| 27 | return; |
| 28 | } |
| 29 | } |
| 30 | } |
| 31 | } |
| 32 | } |
| 33 | |
| 34 | static void test_wacky_bitmapshader(skiatest::Reporter* reporter, |
| 35 | int width, int height, bool shouldBeDrawn) { |
| 36 | SkBitmap dev; |
| 37 | dev.setConfig(SkBitmap::kARGB_8888_Config, 0x56F, 0x4f6); |
| 38 | dev.allocPixels(); |
| 39 | dev.eraseColor(0); // necessary, so we know if we draw to it |
| 40 | |
| 41 | SkMatrix matrix; |
| 42 | |
| 43 | SkCanvas c(dev); |
robertphillips@google.com | 4debcac | 2012-05-14 16:33:36 +0000 | [diff] [blame] | 44 | matrix.setAll(SkFloatToScalar(-119.34097f), |
| 45 | SkFloatToScalar(-43.436558f), |
| 46 | SkFloatToScalar(93489.945f), |
| 47 | SkFloatToScalar(43.436558f), |
| 48 | SkFloatToScalar(-119.34097f), |
| 49 | SkFloatToScalar(123.98426f), |
| 50 | 0, 0, SK_Scalar1); |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 51 | c.concat(matrix); |
| 52 | |
| 53 | SkBitmap bm; |
| 54 | bm.setConfig(SkBitmap::kARGB_8888_Config, width, height); |
| 55 | bm.allocPixels(); |
| 56 | bm.eraseColor(SK_ColorRED); |
| 57 | |
| 58 | SkShader* s = SkShader::CreateBitmapShader(bm, SkShader::kRepeat_TileMode, |
| 59 | SkShader::kRepeat_TileMode); |
robertphillips@google.com | 4debcac | 2012-05-14 16:33:36 +0000 | [diff] [blame] | 60 | matrix.setAll(SkFloatToScalar(0.0078740157f), |
| 61 | 0, |
| 62 | SkIntToScalar(249), |
| 63 | 0, |
| 64 | SkFloatToScalar(0.0078740157f), |
| 65 | SkIntToScalar(239), |
| 66 | 0, 0, SK_Scalar1); |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 67 | s->setLocalMatrix(matrix); |
| 68 | |
| 69 | SkPaint paint; |
| 70 | paint.setShader(s)->unref(); |
| 71 | |
| 72 | SkRect r = SkRect::MakeXYWH(681, 239, 695, 253); |
| 73 | c.drawRect(r, paint); |
| 74 | |
| 75 | assert_ifDrawnTo(reporter, dev, shouldBeDrawn); |
| 76 | } |
reed@google.com | 0da0627 | 2012-05-03 20:26:06 +0000 | [diff] [blame] | 77 | #endif |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 78 | |
| 79 | /* |
| 80 | * Original bug was asserting that the matrix-proc had generated a (Y) value |
| 81 | * that was out of range. This led (in the release build) to the sampler-proc |
| 82 | * reading memory out-of-bounds of the original bitmap. |
| 83 | * |
| 84 | * We were numerically overflowing our 16bit coordinates that we communicate |
| 85 | * between these two procs. The fixes was in two parts: |
| 86 | * |
| 87 | * 1. Just don't draw bitmaps larger than 64K-1 in width or height, since we |
| 88 | * can't represent those coordinates in our transport format (yet). |
| 89 | * 2. Perform an unsigned shift during the calculation, so we don't get |
| 90 | * sign-extension bleed when packing the two values (X,Y) into our 32bit |
| 91 | * slot. |
| 92 | * |
| 93 | * This tests exercises the original setup, plus 3 more to ensure that we can, |
| 94 | * in fact, handle bitmaps at 64K-1 (assuming we don't exceed the total |
| 95 | * memory allocation limit). |
| 96 | */ |
| 97 | static void test_giantrepeat_crbug118018(skiatest::Reporter* reporter) { |
reed@google.com | 0da0627 | 2012-05-03 20:26:06 +0000 | [diff] [blame] | 98 | #ifdef SK_SCALAR_IS_FLOAT |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 99 | static const struct { |
| 100 | int fWidth; |
| 101 | int fHeight; |
| 102 | bool fExpectedToDraw; |
| 103 | } gTests[] = { |
| 104 | { 0x1b294, 0x7f, false }, // crbug 118018 (width exceeds 64K) |
| 105 | { 0xFFFF, 0x7f, true }, // should draw, test max width |
| 106 | { 0x7f, 0xFFFF, true }, // should draw, test max height |
| 107 | { 0xFFFF, 0xFFFF, false }, // allocation fails (too much RAM) |
| 108 | }; |
| 109 | |
| 110 | for (size_t i = 0; i < SK_ARRAY_COUNT(gTests); ++i) { |
| 111 | test_wacky_bitmapshader(reporter, |
| 112 | gTests[i].fWidth, gTests[i].fHeight, |
| 113 | gTests[i].fExpectedToDraw); |
| 114 | } |
reed@google.com | 0da0627 | 2012-05-03 20:26:06 +0000 | [diff] [blame] | 115 | #endif |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 116 | } |
reed@google.com | 0da0627 | 2012-05-03 20:26:06 +0000 | [diff] [blame] | 117 | |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 118 | /////////////////////////////////////////////////////////////////////////////// |
reed@google.com | 2ade086 | 2011-03-17 17:48:04 +0000 | [diff] [blame] | 119 | |
reed@google.com | 6de0bfc | 2012-03-30 17:43:33 +0000 | [diff] [blame] | 120 | static void test_nan_antihair(skiatest::Reporter* reporter) { |
| 121 | SkBitmap bm; |
| 122 | bm.setConfig(SkBitmap::kARGB_8888_Config, 20, 20); |
| 123 | bm.allocPixels(); |
| 124 | |
| 125 | SkCanvas canvas(bm); |
| 126 | |
| 127 | SkPath path; |
| 128 | path.moveTo(0, 0); |
| 129 | path.lineTo(10, SK_ScalarNaN); |
| 130 | |
| 131 | SkPaint paint; |
| 132 | paint.setAntiAlias(true); |
| 133 | paint.setStyle(SkPaint::kStroke_Style); |
| 134 | |
| 135 | // before our fix to SkScan_Antihair.cpp to check for integral NaN (0x800...) |
| 136 | // this would trigger an assert/crash. |
| 137 | // |
| 138 | // see rev. 3558 |
| 139 | canvas.drawPath(path, paint); |
| 140 | } |
| 141 | |
reed@google.com | 2ade086 | 2011-03-17 17:48:04 +0000 | [diff] [blame] | 142 | static bool check_for_all_zeros(const SkBitmap& bm) { |
| 143 | SkAutoLockPixels alp(bm); |
| 144 | |
| 145 | size_t count = bm.width() * bm.bytesPerPixel(); |
| 146 | for (int y = 0; y < bm.height(); y++) { |
| 147 | const uint8_t* ptr = reinterpret_cast<const uint8_t*>(bm.getAddr(0, y)); |
| 148 | for (size_t i = 0; i < count; i++) { |
| 149 | if (ptr[i]) { |
| 150 | return false; |
| 151 | } |
| 152 | } |
| 153 | } |
| 154 | return true; |
| 155 | } |
| 156 | |
| 157 | static const int gWidth = 256; |
| 158 | static const int gHeight = 256; |
| 159 | |
| 160 | static void create(SkBitmap* bm, SkBitmap::Config config, SkColor color) { |
| 161 | bm->setConfig(config, gWidth, gHeight); |
| 162 | bm->allocPixels(); |
| 163 | bm->eraseColor(color); |
| 164 | } |
| 165 | |
| 166 | static void TestDrawBitmapRect(skiatest::Reporter* reporter) { |
| 167 | SkBitmap src, dst; |
| 168 | |
| 169 | create(&src, SkBitmap::kARGB_8888_Config, 0xFFFFFFFF); |
| 170 | create(&dst, SkBitmap::kARGB_8888_Config, 0); |
| 171 | |
| 172 | SkCanvas canvas(dst); |
| 173 | |
| 174 | SkIRect srcR = { gWidth, 0, gWidth + 16, 16 }; |
| 175 | SkRect dstR = { 0, 0, SkIntToScalar(16), SkIntToScalar(16) }; |
| 176 | |
| 177 | canvas.drawBitmapRect(src, &srcR, dstR, NULL); |
| 178 | |
| 179 | // ensure that we draw nothing if srcR does not intersect the bitmap |
| 180 | REPORTER_ASSERT(reporter, check_for_all_zeros(dst)); |
reed@google.com | 6de0bfc | 2012-03-30 17:43:33 +0000 | [diff] [blame] | 181 | |
| 182 | test_nan_antihair(reporter); |
reed@google.com | 99c114e | 2012-05-03 20:14:26 +0000 | [diff] [blame] | 183 | test_giantrepeat_crbug118018(reporter); |
reed@google.com | 2ade086 | 2011-03-17 17:48:04 +0000 | [diff] [blame] | 184 | } |
| 185 | |
| 186 | #include "TestClassDef.h" |
| 187 | DEFINE_TESTCLASS("DrawBitmapRect", TestDrawBitmapRectClass, TestDrawBitmapRect) |