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 | |
| 11 | #include "SkRecord.h" |
| 12 | #include "SkRecordOpts.h" |
| 13 | #include "SkRecorder.h" |
| 14 | #include "SkRecords.h" |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 15 | #include "SkXfermode.h" |
| 16 | |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 17 | static const int W = 1920, H = 1080; |
| 18 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 19 | DEF_TEST(RecordOpts_NoopDraw, r) { |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 20 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 21 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 22 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 23 | recorder.drawRect(SkRect::MakeWH(200, 200), SkPaint()); |
| 24 | recorder.drawRect(SkRect::MakeWH(300, 300), SkPaint()); |
| 25 | recorder.drawRect(SkRect::MakeWH(100, 100), SkPaint()); |
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 | record.replace<SkRecords::NoOp>(1); // NoOps should be allowed. |
commit-bot@chromium.org | 467705a | 2014-05-07 17:17:48 +0000 | [diff] [blame] | 28 | |
| 29 | SkRecordNoopSaveRestores(&record); |
| 30 | |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 31 | 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] | 32 | } |
| 33 | |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 34 | DEF_TEST(RecordOpts_SingleNoopSaveRestore, r) { |
| 35 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 36 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 37 | |
| 38 | recorder.save(); |
| 39 | recorder.clipRect(SkRect::MakeWH(200, 200)); |
| 40 | recorder.restore(); |
| 41 | |
| 42 | SkRecordNoopSaveRestores(&record); |
| 43 | for (unsigned i = 0; i < 3; i++) { |
| 44 | assert_type<SkRecords::NoOp>(r, record, i); |
| 45 | } |
| 46 | } |
| 47 | |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 48 | DEF_TEST(RecordOpts_NoopSaveRestores, r) { |
| 49 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 50 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 51 | |
| 52 | // The second pass will clean up this pair after the first pass noops all the innards. |
| 53 | recorder.save(); |
| 54 | // A simple pointless pair of save/restore. |
| 55 | recorder.save(); |
| 56 | recorder.restore(); |
| 57 | |
| 58 | // As long as we don't draw in there, everything is a noop. |
| 59 | recorder.save(); |
| 60 | recorder.clipRect(SkRect::MakeWH(200, 200)); |
| 61 | recorder.clipRect(SkRect::MakeWH(100, 100)); |
| 62 | recorder.restore(); |
| 63 | recorder.restore(); |
| 64 | |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 65 | SkRecordNoopSaveRestores(&record); |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 66 | for (unsigned index = 0; index < record.count(); index++) { |
commit-bot@chromium.org | 7066bf3 | 2014-05-05 17:09:05 +0000 | [diff] [blame] | 67 | assert_type<SkRecords::NoOp>(r, record, index); |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 68 | } |
commit-bot@chromium.org | 8dac8b1 | 2014-04-30 13:18:12 +0000 | [diff] [blame] | 69 | } |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 70 | |
mtklein | 99d6a9e | 2014-09-10 16:08:27 -0700 | [diff] [blame] | 71 | DEF_TEST(RecordOpts_SaveSaveLayerRestoreRestore, r) { |
| 72 | SkRecord record; |
| 73 | SkRecorder recorder(&record, W, H); |
| 74 | |
| 75 | // A previous bug NoOp'd away the first 3 commands. |
| 76 | recorder.save(); |
| 77 | recorder.saveLayer(NULL, NULL); |
| 78 | recorder.restore(); |
| 79 | recorder.restore(); |
| 80 | |
| 81 | SkRecordNoopSaveRestores(&record); |
reed | 2ff1fce | 2014-12-11 07:07:37 -0800 | [diff] [blame] | 82 | switch (record.count()) { |
| 83 | case 4: |
| 84 | assert_type<SkRecords::Save> (r, record, 0); |
| 85 | assert_type<SkRecords::SaveLayer>(r, record, 1); |
| 86 | assert_type<SkRecords::Restore> (r, record, 2); |
| 87 | assert_type<SkRecords::Restore> (r, record, 3); |
| 88 | break; |
| 89 | case 2: |
| 90 | assert_type<SkRecords::SaveLayer>(r, record, 0); |
| 91 | assert_type<SkRecords::Restore> (r, record, 1); |
| 92 | break; |
| 93 | case 0: |
| 94 | break; |
| 95 | default: |
| 96 | REPORTER_ASSERT(r, false); |
| 97 | } |
mtklein | 99d6a9e | 2014-09-10 16:08:27 -0700 | [diff] [blame] | 98 | } |
| 99 | |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 100 | static void assert_savelayer_restore(skiatest::Reporter* r, |
| 101 | SkRecord* record, |
| 102 | unsigned i, |
| 103 | bool shouldBeNoOped) { |
| 104 | SkRecordNoopSaveLayerDrawRestores(record); |
| 105 | if (shouldBeNoOped) { |
| 106 | assert_type<SkRecords::NoOp>(r, *record, i); |
| 107 | assert_type<SkRecords::NoOp>(r, *record, i+2); |
| 108 | } else { |
| 109 | assert_type<SkRecords::SaveLayer>(r, *record, i); |
| 110 | assert_type<SkRecords::Restore>(r, *record, i+2); |
| 111 | } |
| 112 | } |
| 113 | |
| 114 | DEF_TEST(RecordOpts_NoopSaveLayerDrawRestore, r) { |
| 115 | SkRecord record; |
commit-bot@chromium.org | a095041 | 2014-05-29 16:52:40 +0000 | [diff] [blame] | 116 | SkRecorder recorder(&record, W, H); |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 117 | |
| 118 | SkRect bounds = SkRect::MakeWH(100, 200); |
| 119 | SkRect draw = SkRect::MakeWH(50, 60); |
| 120 | |
| 121 | SkPaint goodLayerPaint, badLayerPaint, worseLayerPaint; |
| 122 | goodLayerPaint.setColor(0x03000000); // Only alpha. |
| 123 | badLayerPaint.setColor( 0x03040506); // Not only alpha. |
| 124 | worseLayerPaint.setXfermodeMode(SkXfermode::kDstIn_Mode); // Any effect will do. |
| 125 | |
| 126 | SkPaint goodDrawPaint, badDrawPaint; |
| 127 | goodDrawPaint.setColor(0xFF020202); // Opaque. |
| 128 | badDrawPaint.setColor( 0x0F020202); // Not opaque. |
| 129 | |
| 130 | // No change: optimization can't handle bounds. |
| 131 | recorder.saveLayer(&bounds, NULL); |
| 132 | recorder.drawRect(draw, goodDrawPaint); |
| 133 | recorder.restore(); |
reed | d990e2f | 2014-12-22 11:58:30 -0800 | [diff] [blame] | 134 | if (!SkCanvas::Internal_Private_GetIgnoreSaveLayerBounds()) { |
| 135 | assert_savelayer_restore(r, &record, 0, false); |
| 136 | } |
commit-bot@chromium.org | f5bf3cf | 2014-05-07 14:47:44 +0000 | [diff] [blame] | 137 | |
| 138 | // SaveLayer/Restore removed: no bounds + no paint = no point. |
| 139 | recorder.saveLayer(NULL, NULL); |
| 140 | recorder.drawRect(draw, goodDrawPaint); |
| 141 | recorder.restore(); |
| 142 | assert_savelayer_restore(r, &record, 3, true); |
| 143 | |
| 144 | // TODO(mtklein): test case with null draw paint |
| 145 | |
| 146 | // No change: layer paint isn't alpha-only. |
| 147 | recorder.saveLayer(NULL, &badLayerPaint); |
| 148 | recorder.drawRect(draw, goodDrawPaint); |
| 149 | recorder.restore(); |
| 150 | assert_savelayer_restore(r, &record, 6, false); |
| 151 | |
| 152 | // No change: layer paint has an effect. |
| 153 | recorder.saveLayer(NULL, &worseLayerPaint); |
| 154 | recorder.drawRect(draw, goodDrawPaint); |
| 155 | recorder.restore(); |
| 156 | assert_savelayer_restore(r, &record, 9, false); |
| 157 | |
| 158 | // No change: draw paint isn't opaque. |
| 159 | recorder.saveLayer(NULL, &goodLayerPaint); |
| 160 | recorder.drawRect(draw, badDrawPaint); |
| 161 | recorder.restore(); |
| 162 | assert_savelayer_restore(r, &record, 12, false); |
| 163 | |
| 164 | // SaveLayer/Restore removed: we can fold in the alpha! |
| 165 | recorder.saveLayer(NULL, &goodLayerPaint); |
| 166 | recorder.drawRect(draw, goodDrawPaint); |
| 167 | recorder.restore(); |
| 168 | assert_savelayer_restore(r, &record, 15, true); |
| 169 | |
| 170 | const SkRecords::DrawRect* drawRect = assert_type<SkRecords::DrawRect>(r, record, 16); |
| 171 | REPORTER_ASSERT(r, drawRect != NULL); |
| 172 | REPORTER_ASSERT(r, drawRect->paint.getColor() == 0x03020202); |
| 173 | } |