scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 1 | |
| 2 | /* |
| 3 | * Copyright 2012 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 | */ |
| 8 | |
scroggo@google.com | 825bb95 | 2012-08-22 15:14:43 +0000 | [diff] [blame] | 9 | #if SK_SUPPORT_GPU |
| 10 | |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 11 | #include "GrContext.h" |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 12 | #include "GrContextFactory.h" |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 13 | #include "SkBitmap.h" |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 14 | #include "SkCanvas.h" |
| 15 | #include "SkColor.h" |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 16 | #include "SkGpuDevice.h" |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 17 | #include "SkPaint.h" |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 18 | #include "SkPixelRef.h" |
| 19 | #include "SkRect.h" |
| 20 | #include "Test.h" |
| 21 | |
| 22 | static const char* boolStr(bool value) { |
| 23 | return value ? "true" : "false"; |
| 24 | } |
| 25 | |
| 26 | // these are in the same order as the SkBitmap::Config enum |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 27 | static const char* gColorTypeName[] = { |
reed@google.com | 6ba4572 | 2013-06-21 18:30:53 +0000 | [diff] [blame] | 28 | "None", "8888" |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 29 | }; |
| 30 | |
| 31 | struct Pair { |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 32 | SkColorType fColorType; |
| 33 | const char* fValid; |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 34 | }; |
| 35 | |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 36 | /** |
| 37 | * Check to ensure that copying a GPU-backed SkBitmap behaved as expected. |
| 38 | * @param reporter Used to report failures. |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 39 | * @param desiredCT colorType being copied to. If the copy succeeded, dst must have this Config. |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 40 | * @param success True if the copy succeeded. |
| 41 | * @param src A GPU-backed SkBitmap that had copyTo or deepCopyTo called on it. |
| 42 | * @param dst SkBitmap that was copied to. |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 43 | * @param expectSameGenID Whether the genIDs should be the same if success is true. |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 44 | */ |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 45 | static void TestIndividualCopy(skiatest::Reporter* reporter, const SkColorType desiredCT, |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 46 | const bool success, const SkBitmap& src, const SkBitmap& dst, |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 47 | const bool expectSameGenID) { |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 48 | if (success) { |
| 49 | REPORTER_ASSERT(reporter, src.width() == dst.width()); |
| 50 | REPORTER_ASSERT(reporter, src.height() == dst.height()); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 51 | REPORTER_ASSERT(reporter, dst.colorType() == desiredCT); |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 52 | if (src.config() == dst.config()) { |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 53 | if (expectSameGenID) { |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 54 | REPORTER_ASSERT(reporter, src.getGenerationID() == dst.getGenerationID()); |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 55 | } else { |
| 56 | REPORTER_ASSERT(reporter, src.getGenerationID() != dst.getGenerationID()); |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 57 | } |
| 58 | REPORTER_ASSERT(reporter, src.pixelRef() != NULL && dst.pixelRef() != NULL); |
| 59 | |
| 60 | // Do read backs and make sure that the two are the same. |
| 61 | SkBitmap srcReadBack, dstReadBack; |
| 62 | { |
| 63 | SkASSERT(src.getTexture() != NULL); |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 64 | const SkIPoint origin = src.pixelRefOrigin(); |
| 65 | const SkIRect subset = SkIRect::MakeXYWH(origin.fX, origin.fY, |
| 66 | src.width(), src.height()); |
| 67 | bool readBack = src.pixelRef()->readPixels(&srcReadBack, &subset); |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 68 | REPORTER_ASSERT(reporter, readBack); |
| 69 | } |
| 70 | if (dst.getTexture() != NULL) { |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 71 | const SkIPoint origin = dst.pixelRefOrigin(); |
| 72 | const SkIRect subset = SkIRect::MakeXYWH(origin.fX, origin.fY, |
| 73 | dst.width(), dst.height()); |
| 74 | bool readBack = dst.pixelRef()->readPixels(&dstReadBack, &subset); |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 75 | REPORTER_ASSERT(reporter, readBack); |
| 76 | } else { |
| 77 | // If dst is not a texture, do a copy instead, to the same config as srcReadBack. |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 78 | bool copy = dst.copyTo(&dstReadBack, srcReadBack.colorType()); |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 79 | REPORTER_ASSERT(reporter, copy); |
| 80 | } |
| 81 | |
| 82 | SkAutoLockPixels srcLock(srcReadBack); |
| 83 | SkAutoLockPixels dstLock(dstReadBack); |
| 84 | REPORTER_ASSERT(reporter, srcReadBack.readyToDraw() && dstReadBack.readyToDraw()); |
| 85 | |
| 86 | const char* srcP = static_cast<const char*>(srcReadBack.getAddr(0, 0)); |
| 87 | const char* dstP = static_cast<const char*>(dstReadBack.getAddr(0, 0)); |
| 88 | REPORTER_ASSERT(reporter, srcP != dstP); |
| 89 | |
| 90 | REPORTER_ASSERT(reporter, !memcmp(srcP, dstP, srcReadBack.getSize())); |
| 91 | } else { |
| 92 | REPORTER_ASSERT(reporter, src.getGenerationID() != dst.getGenerationID()); |
| 93 | } |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 94 | } else { |
| 95 | // dst should be unchanged from its initial state |
| 96 | REPORTER_ASSERT(reporter, dst.config() == SkBitmap::kNo_Config); |
| 97 | REPORTER_ASSERT(reporter, dst.width() == 0); |
| 98 | REPORTER_ASSERT(reporter, dst.height() == 0); |
| 99 | } |
| 100 | |
| 101 | } |
| 102 | |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 103 | // Stripped down version of TestBitmapCopy that checks basic fields (width, height, config, genID) |
| 104 | // to ensure that they were copied properly. |
tfarina@chromium.org | 4ee16bf | 2014-01-10 22:08:27 +0000 | [diff] [blame] | 105 | DEF_GPUTEST(GpuBitmapCopy, reporter, factory) { |
bsalomon@google.com | 7f805ff | 2012-12-10 17:32:07 +0000 | [diff] [blame] | 106 | #ifdef SK_BUILD_FOR_ANDROID // https://code.google.com/p/skia/issues/detail?id=753 |
| 107 | return; |
| 108 | #endif |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 109 | for (int type = 0; type < GrContextFactory::kLastGLContextType; ++type) { |
| 110 | GrContextFactory::GLContextType glType = static_cast<GrContextFactory::GLContextType>(type); |
| 111 | if (!GrContextFactory::IsRenderingGLContext(glType)) { |
| 112 | continue; |
| 113 | } |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 114 | |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 115 | GrContext* grContext = factory->get(glType); |
| 116 | if (NULL == grContext) { |
| 117 | continue; |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 118 | } |
| 119 | |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 120 | |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 121 | if (NULL == grContext) { |
| 122 | return; |
| 123 | } |
| 124 | static const Pair gPairs[] = { |
commit-bot@chromium.org | 15a1405 | 2014-02-16 00:59:25 +0000 | [diff] [blame] | 125 | // SkGpuDevice can no longer be Create()ed with kNo_Config |
| 126 | // (or kUnknown_SkColorType in the new world), hence much of this |
| 127 | // test will be skipped, since it was checking that calling |
| 128 | // copyTo or deepCopyTo with src or dst set to kUnknown/kNo would |
| 129 | // successfully fail. |
| 130 | // |
| 131 | // If we can declare that you can *never* create a texture with |
| 132 | // kUnknown, then perhaps we can remove this entire test... |
| 133 | // |
| 134 | // { SkBitmap::kNo_Config, "00" }, |
| 135 | // { SkBitmap::kARGB_8888_Config, "01" }, |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 136 | { kPMColor_SkColorType, "1" }, |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 137 | }; |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 138 | |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 139 | const int W = 20; |
| 140 | const int H = 33; |
scroggo@google.com | a2a3192 | 2012-12-07 19:14:45 +0000 | [diff] [blame] | 141 | |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 142 | for (size_t i = 0; i < SK_ARRAY_COUNT(gPairs); i++) { |
commit-bot@chromium.org | 15a1405 | 2014-02-16 00:59:25 +0000 | [diff] [blame] | 143 | SkImageInfo info = SkImageInfo::Make(W, H, |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 144 | gPairs[i].fColorType, |
commit-bot@chromium.org | 15a1405 | 2014-02-16 00:59:25 +0000 | [diff] [blame] | 145 | kPremul_SkAlphaType); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 146 | SkBitmap src, dst; |
| 147 | |
commit-bot@chromium.org | 15a1405 | 2014-02-16 00:59:25 +0000 | [diff] [blame] | 148 | SkAutoTUnref<SkGpuDevice> device(SkGpuDevice::Create(grContext, info, 0)); |
| 149 | SkASSERT(device.get()); |
| 150 | |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 151 | src = device->accessBitmap(false); |
| 152 | device->clear(SK_ColorWHITE); |
| 153 | |
| 154 | // Draw something different to the same portion of the bitmap that we will extract as a |
| 155 | // subset, so that comparing the pixels of the subset will be meaningful. |
| 156 | SkIRect subsetRect = SkIRect::MakeLTRB(W/2, H/2, W, H); |
| 157 | SkCanvas drawingCanvas(device); |
| 158 | SkPaint paint; |
| 159 | paint.setColor(SK_ColorRED); |
reed@google.com | 4469938 | 2013-10-31 17:28:30 +0000 | [diff] [blame] | 160 | drawingCanvas.drawRect(SkRect::Make(subsetRect), paint); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 161 | |
| 162 | // Extract a subset. If this succeeds we will test copying the subset. |
| 163 | SkBitmap subset; |
commit-bot@chromium.org | 6285f4f | 2014-02-20 19:08:07 +0000 | [diff] [blame] | 164 | #ifdef SK_SUPPORT_DEEPCOPYTO_CONFIG |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 165 | const bool extracted = src.extractSubset(&subset, subsetRect); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 166 | |
| 167 | for (size_t j = 0; j < SK_ARRAY_COUNT(gPairs); j++) { |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 168 | SkBitmap::Config pairsConfig = SkColorTypeToBitmapConfig(gPairs[j].fColorType); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 169 | dst.reset(); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 170 | bool success = src.deepCopyTo(&dst, pairsConfig); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 171 | bool expected = gPairs[i].fValid[j] != '0'; |
| 172 | if (success != expected) { |
halcanary@google.com | a9325fa | 2014-01-10 14:58:10 +0000 | [diff] [blame] | 173 | ERRORF(reporter, "SkBitmap::deepCopyTo from %s to %s. " |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 174 | "expected %s returned %s", gColorTypeName[i], |
| 175 | gColorTypeName[j], boolStr(expected), |
halcanary@google.com | a9325fa | 2014-01-10 14:58:10 +0000 | [diff] [blame] | 176 | boolStr(success)); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 177 | } |
| 178 | |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 179 | bool canSucceed = src.canCopyTo(gPairs[j].fColorType); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 180 | if (success != canSucceed) { |
halcanary@google.com | a9325fa | 2014-01-10 14:58:10 +0000 | [diff] [blame] | 181 | ERRORF(reporter, "SkBitmap::deepCopyTo from %s to %s " |
| 182 | "returned %s, but canCopyTo returned %s", |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 183 | gColorTypeName[i], gColorTypeName[j], boolStr(success), |
halcanary@google.com | a9325fa | 2014-01-10 14:58:10 +0000 | [diff] [blame] | 184 | boolStr(canSucceed)); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 185 | } |
| 186 | |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 187 | TestIndividualCopy(reporter, gPairs[j].fColorType, success, src, dst, true); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 188 | |
| 189 | // Test copying the subset bitmap, using both copyTo and deepCopyTo. |
| 190 | if (extracted) { |
| 191 | SkBitmap subsetCopy; |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 192 | success = subset.copyTo(&subsetCopy, gPairs[j].fColorType); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 193 | REPORTER_ASSERT(reporter, success == expected); |
| 194 | REPORTER_ASSERT(reporter, success == canSucceed); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 195 | TestIndividualCopy(reporter, gPairs[j].fColorType, success, subset, subsetCopy, |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 196 | true); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 197 | |
| 198 | // Reset the bitmap so that a failed copyTo will leave it in the expected state. |
| 199 | subsetCopy.reset(); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 200 | success = subset.deepCopyTo(&subsetCopy, pairsConfig); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 201 | REPORTER_ASSERT(reporter, success == expected); |
| 202 | REPORTER_ASSERT(reporter, success == canSucceed); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 203 | TestIndividualCopy(reporter, gPairs[j].fColorType, success, subset, subsetCopy, |
scroggo@google.com | 9349101 | 2013-02-05 19:22:27 +0000 | [diff] [blame] | 204 | true); |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 205 | |
| 206 | // Now set a bitmap to be a subset that will share the same pixelref. |
| 207 | // This allows testing another case of cloning the genID. When calling copyTo |
| 208 | // on a bitmap representing a subset of its pixelref, the resulting pixelref |
| 209 | // should not share the genID, since we only copied the subset. |
| 210 | SkBitmap trueSubset; |
| 211 | // FIXME: Once https://codereview.chromium.org/109023008/ lands, call |
| 212 | // trueSubset.installPixelRef(src.pixelRef(), subset); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 213 | trueSubset.setConfig(SkImageInfo::Make(W/2, H/2, gPairs[i].fColorType, |
| 214 | kPremul_SkAlphaType)); |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 215 | trueSubset.setPixelRef(src.pixelRef(), W/2, H/2); |
| 216 | |
| 217 | subsetCopy.reset(); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 218 | success = trueSubset.copyTo(&subsetCopy, gPairs[j].fColorType); |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 219 | REPORTER_ASSERT(reporter, success == expected); |
| 220 | REPORTER_ASSERT(reporter, success == canSucceed); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 221 | TestIndividualCopy(reporter, gPairs[j].fColorType, success, trueSubset, subsetCopy, |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 222 | false); |
| 223 | |
| 224 | // deepCopyTo copies the entire pixelref, even if the bitmap only represents |
| 225 | // a subset. Therefore, the result should share the same genID. |
| 226 | subsetCopy.reset(); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 227 | success = trueSubset.deepCopyTo(&subsetCopy, pairsConfig); |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 228 | REPORTER_ASSERT(reporter, success == expected); |
| 229 | REPORTER_ASSERT(reporter, success == canSucceed); |
commit-bot@chromium.org | 8a2ad3c | 2014-02-23 03:59:35 +0000 | [diff] [blame^] | 230 | TestIndividualCopy(reporter, gPairs[j].fColorType, success, trueSubset, subsetCopy, |
scroggo@google.com | 5ccae2c | 2014-01-15 16:56:52 +0000 | [diff] [blame] | 231 | true); |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 232 | } |
| 233 | } // for (size_t j = ... |
commit-bot@chromium.org | 6285f4f | 2014-02-20 19:08:07 +0000 | [diff] [blame] | 234 | #endif |
bsalomon@google.com | 67b915d | 2013-02-04 16:13:32 +0000 | [diff] [blame] | 235 | } // for (size_t i = ... |
| 236 | } // GrContextFactory::GLContextType |
scroggo@google.com | d5764e8 | 2012-08-22 15:00:05 +0000 | [diff] [blame] | 237 | } |
| 238 | |
scroggo@google.com | 825bb95 | 2012-08-22 15:14:43 +0000 | [diff] [blame] | 239 | #endif |