commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2014 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 | */ |
| 7 | |
| 8 | #include "Test.h" |
commit-bot@chromium.org | 0a98d87 | 2014-05-19 15:15:24 +0000 | [diff] [blame] | 9 | #include "RecordTestUtils.h" |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 10 | |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 11 | #include "SkColorFilter.h" |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 12 | #include "SkRecord.h" |
| 13 | #include "SkRecordOpts.h" |
| 14 | #include "SkRecorder.h" |
| 15 | #include "SkRecords.h" |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 16 | #include "SkXfermode.h" |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 17 | #include "SkPictureRecorder.h" |
| 18 | #include "SkPictureImageFilter.h" |
reed | 5a3c528 | 2016-09-06 06:54:41 -0700 | [diff] [blame] | 19 | #include "SkSurface.h" |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 20 | |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 21 | static const int W = 1920, H = 1080; |
| 22 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 23 | DEF_TEST(RecordOpts_NoopDraw, r) { |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 24 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 25 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 26 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 27 | recorder.drawRect(SkRect::MakeWH(200, 200), SkPaint()); |
| 28 | recorder.drawRect(SkRect::MakeWH(300, 300), SkPaint()); |
| 29 | recorder.drawRect(SkRect::MakeWH(100, 100), SkPaint()); |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 30 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 31 | record.replace<SkRecords::NoOp>(1); // NoOps should be allowed. |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 32 | |
| 33 | SkRecordNoopSaveRestores(&record); |
| 34 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 35 | REPORTER_ASSERT(r, 2 == count_instances_of_type<SkRecords::DrawRect>(record)); |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 36 | } |
| 37 | |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 38 | DEF_TEST(RecordOpts_SingleNoopSaveRestore, r) { |
| 39 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 40 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 41 | |
| 42 | recorder.save(); |
| 43 | recorder.clipRect(SkRect::MakeWH(200, 200)); |
| 44 | recorder.restore(); |
| 45 | |
| 46 | SkRecordNoopSaveRestores(&record); |
mtklein | c6ad06a | 2015-08-19 09:51:00 -0700 | [diff] [blame] | 47 | for (int i = 0; i < 3; i++) { |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 48 | assert_type<SkRecords::NoOp>(r, record, i); |
| 49 | } |
| 50 | } |
| 51 | |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 52 | DEF_TEST(RecordOpts_NoopSaveRestores, r) { |
| 53 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 54 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 55 | |
| 56 | // The second pass will clean up this pair after the first pass noops all the innards. |
| 57 | recorder.save(); |
| 58 | // A simple pointless pair of save/restore. |
| 59 | recorder.save(); |
| 60 | recorder.restore(); |
| 61 | |
| 62 | // As long as we don't draw in there, everything is a noop. |
| 63 | recorder.save(); |
| 64 | recorder.clipRect(SkRect::MakeWH(200, 200)); |
| 65 | recorder.clipRect(SkRect::MakeWH(100, 100)); |
| 66 | recorder.restore(); |
| 67 | recorder.restore(); |
| 68 | |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 69 | SkRecordNoopSaveRestores(&record); |
mtklein | c6ad06a | 2015-08-19 09:51:00 -0700 | [diff] [blame] | 70 | for (int index = 0; index < record.count(); index++) { |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 71 | assert_type<SkRecords::NoOp>(r, record, index); |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 72 | } |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 73 | } |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 74 | |
mtklein | 99d6a9e | 2014-09-10 16:08:27 -0700 | [diff] [blame] | 75 | DEF_TEST(RecordOpts_SaveSaveLayerRestoreRestore, r) { |
| 76 | SkRecord record; |
| 77 | SkRecorder recorder(&record, W, H); |
| 78 | |
| 79 | // A previous bug NoOp'd away the first 3 commands. |
| 80 | recorder.save(); |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 81 | recorder.saveLayer(nullptr, nullptr); |
mtklein | 99d6a9e | 2014-09-10 16:08:27 -0700 | [diff] [blame] | 82 | recorder.restore(); |
| 83 | recorder.restore(); |
| 84 | |
| 85 | SkRecordNoopSaveRestores(&record); |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 86 | switch (record.count()) { |
| 87 | case 4: |
| 88 | assert_type<SkRecords::Save> (r, record, 0); |
| 89 | assert_type<SkRecords::SaveLayer>(r, record, 1); |
| 90 | assert_type<SkRecords::Restore> (r, record, 2); |
| 91 | assert_type<SkRecords::Restore> (r, record, 3); |
| 92 | break; |
| 93 | case 2: |
| 94 | assert_type<SkRecords::SaveLayer>(r, record, 0); |
| 95 | assert_type<SkRecords::Restore> (r, record, 1); |
| 96 | break; |
| 97 | case 0: |
| 98 | break; |
| 99 | default: |
| 100 | REPORTER_ASSERT(r, false); |
| 101 | } |
mtklein | 99d6a9e | 2014-09-10 16:08:27 -0700 | [diff] [blame] | 102 | } |
| 103 | |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 104 | static void assert_savelayer_restore(skiatest::Reporter* r, |
| 105 | SkRecord* record, |
mtklein | c6ad06a | 2015-08-19 09:51:00 -0700 | [diff] [blame] | 106 | int i, |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 107 | bool shouldBeNoOped) { |
| 108 | SkRecordNoopSaveLayerDrawRestores(record); |
| 109 | if (shouldBeNoOped) { |
| 110 | assert_type<SkRecords::NoOp>(r, *record, i); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 111 | assert_type<SkRecords::NoOp>(r, *record, i+1); |
| 112 | } else { |
| 113 | assert_type<SkRecords::SaveLayer>(r, *record, i); |
| 114 | assert_type<SkRecords::Restore>(r, *record, i+1); |
| 115 | } |
| 116 | } |
| 117 | |
| 118 | static void assert_savelayer_draw_restore(skiatest::Reporter* r, |
| 119 | SkRecord* record, |
| 120 | int i, |
| 121 | bool shouldBeNoOped) { |
| 122 | SkRecordNoopSaveLayerDrawRestores(record); |
| 123 | if (shouldBeNoOped) { |
| 124 | assert_type<SkRecords::NoOp>(r, *record, i); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 125 | assert_type<SkRecords::NoOp>(r, *record, i+2); |
| 126 | } else { |
| 127 | assert_type<SkRecords::SaveLayer>(r, *record, i); |
| 128 | assert_type<SkRecords::Restore>(r, *record, i+2); |
| 129 | } |
| 130 | } |
| 131 | |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 132 | #include "SkBlurImageFilter.h" |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 133 | DEF_TEST(RecordOpts_NoopSaveLayerDrawRestore, r) { |
| 134 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 135 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 136 | |
| 137 | SkRect bounds = SkRect::MakeWH(100, 200); |
| 138 | SkRect draw = SkRect::MakeWH(50, 60); |
| 139 | |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 140 | SkPaint alphaOnlyLayerPaint, translucentLayerPaint, xfermodeLayerPaint; |
| 141 | alphaOnlyLayerPaint.setColor(0x03000000); // Only alpha. |
| 142 | translucentLayerPaint.setColor(0x03040506); // Not only alpha. |
| 143 | xfermodeLayerPaint.setXfermodeMode(SkXfermode::kDstIn_Mode); // Any effect will do. |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 144 | |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 145 | SkPaint opaqueDrawPaint, translucentDrawPaint; |
| 146 | opaqueDrawPaint.setColor(0xFF020202); // Opaque. |
| 147 | translucentDrawPaint.setColor(0x0F020202); // Not opaque. |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 148 | |
mtklein | 0c27897 | 2015-01-15 08:08:10 -0800 | [diff] [blame] | 149 | // SaveLayer/Restore removed: No paint = no point. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 150 | recorder.saveLayer(nullptr, nullptr); |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 151 | recorder.drawRect(draw, opaqueDrawPaint); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 152 | recorder.restore(); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 153 | assert_savelayer_draw_restore(r, &record, 0, true); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 154 | |
mtklein | 0c27897 | 2015-01-15 08:08:10 -0800 | [diff] [blame] | 155 | // Bounds don't matter. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 156 | recorder.saveLayer(&bounds, nullptr); |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 157 | recorder.drawRect(draw, opaqueDrawPaint); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 158 | recorder.restore(); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 159 | assert_savelayer_draw_restore(r, &record, 3, true); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 160 | |
| 161 | // TODO(mtklein): test case with null draw paint |
| 162 | |
| 163 | // No change: layer paint isn't alpha-only. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 164 | recorder.saveLayer(nullptr, &translucentLayerPaint); |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 165 | recorder.drawRect(draw, opaqueDrawPaint); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 166 | recorder.restore(); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 167 | assert_savelayer_draw_restore(r, &record, 6, false); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 168 | |
| 169 | // No change: layer paint has an effect. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 170 | recorder.saveLayer(nullptr, &xfermodeLayerPaint); |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 171 | recorder.drawRect(draw, opaqueDrawPaint); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 172 | recorder.restore(); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 173 | assert_savelayer_draw_restore(r, &record, 9, false); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 174 | |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 175 | // SaveLayer/Restore removed: we can fold in the alpha! |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 176 | recorder.saveLayer(nullptr, &alphaOnlyLayerPaint); |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 177 | recorder.drawRect(draw, translucentDrawPaint); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 178 | recorder.restore(); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 179 | assert_savelayer_draw_restore(r, &record, 12, true); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 180 | |
| 181 | // SaveLayer/Restore removed: we can fold in the alpha! |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 182 | recorder.saveLayer(nullptr, &alphaOnlyLayerPaint); |
kkinnunen | 678c1b0 | 2015-01-16 05:04:36 -0800 | [diff] [blame] | 183 | recorder.drawRect(draw, opaqueDrawPaint); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 184 | recorder.restore(); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 185 | assert_savelayer_draw_restore(r, &record, 15, true); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 186 | |
| 187 | const SkRecords::DrawRect* drawRect = assert_type<SkRecords::DrawRect>(r, record, 16); |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 188 | REPORTER_ASSERT(r, drawRect != nullptr); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 189 | REPORTER_ASSERT(r, drawRect->paint.getColor() == 0x03020202); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 190 | |
| 191 | // saveLayer w/ backdrop should NOT go away |
robertphillips | 6e7025a | 2016-04-04 04:31:25 -0700 | [diff] [blame] | 192 | sk_sp<SkImageFilter> filter(SkBlurImageFilter::Make(3, 3, nullptr)); |
| 193 | recorder.saveLayer({ nullptr, nullptr, filter.get(), 0}); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 194 | recorder.drawRect(draw, opaqueDrawPaint); |
| 195 | recorder.restore(); |
| 196 | assert_savelayer_draw_restore(r, &record, 18, false); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 197 | } |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 198 | |
| 199 | static void assert_merge_svg_opacity_and_filter_layers(skiatest::Reporter* r, |
| 200 | SkRecord* record, |
mtklein | c6ad06a | 2015-08-19 09:51:00 -0700 | [diff] [blame] | 201 | int i, |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 202 | bool shouldBeNoOped) { |
| 203 | SkRecordMergeSvgOpacityAndFilterLayers(record); |
| 204 | if (shouldBeNoOped) { |
| 205 | assert_type<SkRecords::NoOp>(r, *record, i); |
| 206 | assert_type<SkRecords::NoOp>(r, *record, i + 6); |
| 207 | } else { |
| 208 | assert_type<SkRecords::SaveLayer>(r, *record, i); |
| 209 | assert_type<SkRecords::Restore>(r, *record, i + 6); |
| 210 | } |
| 211 | } |
| 212 | |
| 213 | DEF_TEST(RecordOpts_MergeSvgOpacityAndFilterLayers, r) { |
| 214 | SkRecord record; |
| 215 | SkRecorder recorder(&record, W, H); |
| 216 | |
| 217 | SkRect bounds = SkRect::MakeWH(SkIntToScalar(100), SkIntToScalar(200)); |
| 218 | SkRect clip = SkRect::MakeWH(SkIntToScalar(50), SkIntToScalar(60)); |
| 219 | |
| 220 | SkPaint alphaOnlyLayerPaint; |
| 221 | alphaOnlyLayerPaint.setColor(0x03000000); // Only alpha. |
| 222 | SkPaint translucentLayerPaint; |
| 223 | translucentLayerPaint.setColor(0x03040506); // Not only alpha. |
| 224 | SkPaint xfermodePaint; |
| 225 | xfermodePaint.setXfermodeMode(SkXfermode::kDstIn_Mode); |
| 226 | SkPaint colorFilterPaint; |
| 227 | colorFilterPaint.setColorFilter( |
reed | d053ce9 | 2016-03-22 10:17:23 -0700 | [diff] [blame] | 228 | SkColorFilter::MakeModeFilter(SK_ColorLTGRAY, SkXfermode::kSrcIn_Mode)); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 229 | |
| 230 | SkPaint opaqueFilterLayerPaint; |
| 231 | opaqueFilterLayerPaint.setColor(0xFF020202); // Opaque. |
| 232 | SkPaint translucentFilterLayerPaint; |
| 233 | translucentFilterLayerPaint.setColor(0x0F020202); // Not opaque. |
reed | ca2622b | 2016-03-18 07:25:55 -0700 | [diff] [blame] | 234 | sk_sp<SkPicture> shape; |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 235 | { |
| 236 | SkPictureRecorder recorder; |
| 237 | SkCanvas* canvas = recorder.beginRecording(SkIntToScalar(100), SkIntToScalar(100)); |
| 238 | SkPaint shapePaint; |
| 239 | shapePaint.setColor(SK_ColorWHITE); |
| 240 | canvas->drawRect(SkRect::MakeWH(SkIntToScalar(50), SkIntToScalar(50)), shapePaint); |
reed | ca2622b | 2016-03-18 07:25:55 -0700 | [diff] [blame] | 241 | shape = recorder.finishRecordingAsPicture(); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 242 | } |
robertphillips | 5ff17b1 | 2016-03-28 13:13:42 -0700 | [diff] [blame] | 243 | translucentFilterLayerPaint.setImageFilter(SkPictureImageFilter::Make(shape)); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 244 | |
| 245 | int index = 0; |
| 246 | |
| 247 | { |
robertphillips | 6e7025a | 2016-04-04 04:31:25 -0700 | [diff] [blame] | 248 | sk_sp<SkImageFilter> filter(SkBlurImageFilter::Make(3, 3, nullptr)); |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 249 | // first (null) should be optimized, 2nd should not |
| 250 | SkImageFilter* filters[] = { nullptr, filter.get() }; |
| 251 | |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 252 | // Any combination of these should cause the pattern to be optimized. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 253 | SkRect* firstBounds[] = { nullptr, &bounds }; |
| 254 | SkPaint* firstPaints[] = { nullptr, &alphaOnlyLayerPaint }; |
| 255 | SkRect* secondBounds[] = { nullptr, &bounds }; |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 256 | SkPaint* secondPaints[] = { &opaqueFilterLayerPaint, &translucentFilterLayerPaint }; |
| 257 | |
reed | bfd5f17 | 2016-01-07 11:28:08 -0800 | [diff] [blame] | 258 | for (auto outerF : filters) { |
| 259 | bool outerNoOped = !outerF; |
| 260 | for (auto innerF : filters) { |
| 261 | for (size_t i = 0; i < SK_ARRAY_COUNT(firstBounds); ++ i) { |
| 262 | for (size_t j = 0; j < SK_ARRAY_COUNT(firstPaints); ++j) { |
| 263 | for (size_t k = 0; k < SK_ARRAY_COUNT(secondBounds); ++k) { |
| 264 | for (size_t m = 0; m < SK_ARRAY_COUNT(secondPaints); ++m) { |
| 265 | bool innerNoOped = !secondBounds[k] && !secondPaints[m] && !innerF; |
| 266 | |
| 267 | recorder.saveLayer({firstBounds[i], firstPaints[j], outerF, 0}); |
| 268 | recorder.save(); |
| 269 | recorder.clipRect(clip); |
| 270 | recorder.saveLayer({secondBounds[k], secondPaints[m], innerF, 0}); |
| 271 | recorder.restore(); |
| 272 | recorder.restore(); |
| 273 | recorder.restore(); |
| 274 | assert_merge_svg_opacity_and_filter_layers(r, &record, index, |
| 275 | outerNoOped); |
| 276 | assert_savelayer_restore(r, &record, index + 3, innerNoOped); |
| 277 | index += 7; |
| 278 | } |
| 279 | } |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 280 | } |
| 281 | } |
| 282 | } |
| 283 | } |
| 284 | } |
| 285 | |
| 286 | // These should cause the pattern to stay unoptimized: |
| 287 | struct { |
| 288 | SkPaint* firstPaint; |
| 289 | SkPaint* secondPaint; |
| 290 | } noChangeTests[] = { |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 291 | // No change: nullptr filter layer paint not implemented. |
| 292 | { &alphaOnlyLayerPaint, nullptr }, |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 293 | // No change: layer paint is not alpha-only. |
| 294 | { &translucentLayerPaint, &opaqueFilterLayerPaint }, |
| 295 | // No change: layer paint has an xfereffect. |
| 296 | { &xfermodePaint, &opaqueFilterLayerPaint }, |
| 297 | // No change: filter layer paint has an xfereffect. |
| 298 | { &alphaOnlyLayerPaint, &xfermodePaint }, |
| 299 | // No change: layer paint has a color filter. |
| 300 | { &colorFilterPaint, &opaqueFilterLayerPaint }, |
| 301 | // No change: filter layer paint has a color filter (until the optimization accounts for |
| 302 | // constant color draws that can filter the color). |
| 303 | { &alphaOnlyLayerPaint, &colorFilterPaint } |
| 304 | }; |
| 305 | |
| 306 | for (size_t i = 0; i < SK_ARRAY_COUNT(noChangeTests); ++i) { |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 307 | recorder.saveLayer(nullptr, noChangeTests[i].firstPaint); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 308 | recorder.save(); |
| 309 | recorder.clipRect(clip); |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 310 | recorder.saveLayer(nullptr, noChangeTests[i].secondPaint); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 311 | recorder.restore(); |
| 312 | recorder.restore(); |
| 313 | recorder.restore(); |
| 314 | assert_merge_svg_opacity_and_filter_layers(r, &record, index, false); |
| 315 | index += 7; |
| 316 | } |
| 317 | |
| 318 | // Test the folded alpha value. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 319 | recorder.saveLayer(nullptr, &alphaOnlyLayerPaint); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 320 | recorder.save(); |
| 321 | recorder.clipRect(clip); |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 322 | recorder.saveLayer(nullptr, &opaqueFilterLayerPaint); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 323 | recorder.restore(); |
| 324 | recorder.restore(); |
| 325 | recorder.restore(); |
| 326 | assert_merge_svg_opacity_and_filter_layers(r, &record, index, true); |
| 327 | |
| 328 | const SkRecords::SaveLayer* saveLayer = assert_type<SkRecords::SaveLayer>(r, record, index + 3); |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 329 | REPORTER_ASSERT(r, saveLayer != nullptr); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 330 | REPORTER_ASSERT(r, saveLayer->paint->getColor() == 0x03020202); |
| 331 | |
| 332 | index += 7; |
| 333 | |
| 334 | // Test that currently we do not fold alphas for patterns without the clip. This is just not |
| 335 | // implemented. |
halcanary | 96fcdcc | 2015-08-27 07:41:13 -0700 | [diff] [blame] | 336 | recorder.saveLayer(nullptr, &alphaOnlyLayerPaint); |
| 337 | recorder.saveLayer(nullptr, &opaqueFilterLayerPaint); |
kkinnunen | dc0f408 | 2015-01-26 00:14:26 -0800 | [diff] [blame] | 338 | recorder.restore(); |
| 339 | recorder.restore(); |
| 340 | SkRecordMergeSvgOpacityAndFilterLayers(&record); |
| 341 | assert_type<SkRecords::SaveLayer>(r, record, index); |
| 342 | assert_type<SkRecords::SaveLayer>(r, record, index + 1); |
| 343 | assert_type<SkRecords::Restore>(r, record, index + 2); |
| 344 | assert_type<SkRecords::Restore>(r, record, index + 3); |
| 345 | index += 4; |
| 346 | } |
reed | 5a3c528 | 2016-09-06 06:54:41 -0700 | [diff] [blame] | 347 | |
| 348 | static void do_draw(SkCanvas* canvas, SkColor color, bool doLayer) { |
| 349 | canvas->drawColor(SK_ColorWHITE); |
| 350 | |
| 351 | SkPaint p; |
| 352 | p.setColor(color); |
| 353 | |
| 354 | if (doLayer) { |
| 355 | canvas->saveLayer(nullptr, nullptr); |
| 356 | p.setXfermodeMode(SkXfermode::kSrc_Mode); |
| 357 | canvas->drawPaint(p); |
| 358 | canvas->restore(); |
| 359 | } else { |
| 360 | canvas->drawPaint(p); |
| 361 | } |
| 362 | } |
| 363 | |
| 364 | static bool is_equal(SkSurface* a, SkSurface* b) { |
| 365 | const SkImageInfo info = SkImageInfo::MakeN32Premul(1, 1); |
| 366 | SkPMColor ca, cb; |
| 367 | a->readPixels(info, &ca, sizeof(SkPMColor), 0, 0); |
| 368 | b->readPixels(info, &cb, sizeof(SkPMColor), 0, 0); |
| 369 | return ca == cb; |
| 370 | } |
| 371 | |
| 372 | // Test drawing w/ and w/o a simple layer (no bounds or paint), so see that drawing ops |
| 373 | // that *should* draw the same in fact do. |
| 374 | // |
| 375 | // Perform this test twice : once directly, and once via a picture |
| 376 | // |
| 377 | static void do_savelayer_srcmode(skiatest::Reporter* r, SkColor color) { |
| 378 | for (int doPicture = 0; doPicture <= 1; ++doPicture) { |
| 379 | sk_sp<SkSurface> surf0 = SkSurface::MakeRasterN32Premul(10, 10); |
| 380 | sk_sp<SkSurface> surf1 = SkSurface::MakeRasterN32Premul(10, 10); |
| 381 | SkCanvas* c0 = surf0->getCanvas(); |
| 382 | SkCanvas* c1 = surf1->getCanvas(); |
| 383 | |
| 384 | SkPictureRecorder rec0, rec1; |
| 385 | if (doPicture) { |
| 386 | c0 = rec0.beginRecording(10, 10); |
| 387 | c1 = rec1.beginRecording(10, 10); |
| 388 | } |
| 389 | |
| 390 | do_draw(c0, color, false); |
| 391 | do_draw(c1, color, true); |
| 392 | |
| 393 | if (doPicture) { |
| 394 | surf0->getCanvas()->drawPicture(rec0.finishRecordingAsPicture()); |
| 395 | surf1->getCanvas()->drawPicture(rec1.finishRecordingAsPicture()); |
| 396 | } |
| 397 | |
| 398 | // we replicate the assert so we can see which line is reported if there is a failure |
| 399 | if (doPicture) { |
| 400 | REPORTER_ASSERT(r, is_equal(surf0.get(), surf1.get())); |
| 401 | } else { |
| 402 | REPORTER_ASSERT(r, is_equal(surf0.get(), surf1.get())); |
| 403 | } |
| 404 | } |
| 405 | } |
| 406 | |
| 407 | DEF_TEST(savelayer_srcmode_opaque, r) { |
| 408 | do_savelayer_srcmode(r, SK_ColorRED); |
| 409 | } |
| 410 | |
| 411 | DEF_TEST(savelayer_srcmode_alpha, r) { |
| 412 | do_savelayer_srcmode(r, 0x80FF0000); |
| 413 | } |
| 414 | |