blob: 49444f37e0a63a7403035a64d20d0a5612b1f5ee [file] [log] [blame]
commit-bot@chromium.orgc4b21e62014-04-11 18:33:31 +00001/*
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
Mike Kleinc0bd9f92019-04-23 12:05:21 -05008#include "tests/RecordTestUtils.h"
9#include "tests/Test.h"
commit-bot@chromium.orgd9ce2be2014-04-09 23:30:28 +000010
Mike Kleinc0bd9f92019-04-23 12:05:21 -050011#include "include/core/SkSurface.h"
Michael Ludwig55edb502019-08-05 10:41:10 -040012#include "include/effects/SkImageFilters.h"
Mike Kleinc0bd9f92019-04-23 12:05:21 -050013#include "src/core/SkImagePriv.h"
14#include "src/core/SkRecord.h"
15#include "src/core/SkRecordDraw.h"
16#include "src/core/SkRecordOpts.h"
17#include "src/core/SkRecorder.h"
18#include "src/core/SkRecords.h"
19#include "tools/debugger/DebugCanvas.h"
commit-bot@chromium.orgd9ce2be2014-04-09 23:30:28 +000020
21static const int W = 1920, H = 1080;
22
robertphillips783fe162015-01-07 07:28:41 -080023class JustOneDraw : public SkPicture::AbortCallback {
Mike Kleinc11530e2014-06-24 11:29:06 -040024public:
25 JustOneDraw() : fCalls(0) {}
26
mtklein36352bf2015-03-25 18:17:31 -070027 bool abort() override { return fCalls++ > 0; }
Mike Kleinc11530e2014-06-24 11:29:06 -040028private:
29 int fCalls;
30};
31
reed2ff1fce2014-12-11 07:07:37 -080032DEF_TEST(RecordDraw_LazySaves, r) {
33 // Record two commands.
34 SkRecord record;
35 SkRecorder recorder(&record, W, H);
36
37 REPORTER_ASSERT(r, 0 == record.count());
38 recorder.save();
39 REPORTER_ASSERT(r, 0 == record.count()); // the save was not recorded (yet)
40 recorder.drawColor(SK_ColorRED);
41 REPORTER_ASSERT(r, 1 == record.count());
42 recorder.scale(2, 2);
43 REPORTER_ASSERT(r, 3 == record.count()); // now we see the save
44 recorder.restore();
45 REPORTER_ASSERT(r, 4 == record.count());
46
47 assert_type<SkRecords::DrawPaint>(r, record, 0);
48 assert_type<SkRecords::Save> (r, record, 1);
Mike Reed9403c382020-01-13 14:40:56 +000049 assert_type<SkRecords::Scale> (r, record, 2);
reed2ff1fce2014-12-11 07:07:37 -080050 assert_type<SkRecords::Restore> (r, record, 3);
51
52 recorder.save();
53 recorder.save();
54 recorder.restore();
55 recorder.restore();
56 REPORTER_ASSERT(r, 4 == record.count());
57}
58
Mike Kleinc11530e2014-06-24 11:29:06 -040059DEF_TEST(RecordDraw_Abort, r) {
60 // Record two commands.
61 SkRecord record;
62 SkRecorder recorder(&record, W, H);
63 recorder.drawRect(SkRect::MakeWH(200, 300), SkPaint());
64 recorder.clipRect(SkRect::MakeWH(100, 200));
65
66 SkRecord rerecord;
67 SkRecorder canvas(&rerecord, W, H);
68
69 JustOneDraw callback;
halcanary96fcdcc2015-08-27 07:41:13 -070070 SkRecordDraw(record, &canvas, nullptr, nullptr, 0, nullptr/*bbh*/, &callback);
Mike Kleinc11530e2014-06-24 11:29:06 -040071
reed2ff1fce2014-12-11 07:07:37 -080072 REPORTER_ASSERT(r, 1 == count_instances_of_type<SkRecords::DrawRect>(rerecord));
73 REPORTER_ASSERT(r, 0 == count_instances_of_type<SkRecords::ClipRect>(rerecord));
Mike Kleinc11530e2014-06-24 11:29:06 -040074}
75
76DEF_TEST(RecordDraw_Unbalanced, r) {
77 SkRecord record;
78 SkRecorder recorder(&record, W, H);
79 recorder.save(); // We won't balance this, but SkRecordDraw will for us.
reed2ff1fce2014-12-11 07:07:37 -080080 recorder.scale(2, 2);
Mike Kleinc11530e2014-06-24 11:29:06 -040081
82 SkRecord rerecord;
83 SkRecorder canvas(&rerecord, W, H);
halcanary96fcdcc2015-08-27 07:41:13 -070084 SkRecordDraw(record, &canvas, nullptr, nullptr, 0, nullptr/*bbh*/, nullptr/*callback*/);
Mike Kleinc11530e2014-06-24 11:29:06 -040085
reed2ff1fce2014-12-11 07:07:37 -080086 int save_count = count_instances_of_type<SkRecords::Save>(rerecord);
87 int restore_count = count_instances_of_type<SkRecords::Save>(rerecord);
88 REPORTER_ASSERT(r, save_count == restore_count);
Mike Kleinc11530e2014-06-24 11:29:06 -040089}
90
commit-bot@chromium.org0a98d872014-05-19 15:15:24 +000091DEF_TEST(RecordDraw_SetMatrixClobber, r) {
92 // Set up an SkRecord that just scales by 2x,3x.
93 SkRecord scaleRecord;
commit-bot@chromium.orga0950412014-05-29 16:52:40 +000094 SkRecorder scaleCanvas(&scaleRecord, W, H);
commit-bot@chromium.org0a98d872014-05-19 15:15:24 +000095 SkMatrix scale;
96 scale.setScale(2, 3);
97 scaleCanvas.setMatrix(scale);
98
99 // Set up an SkRecord with an initial +20, +20 translate.
100 SkRecord translateRecord;
commit-bot@chromium.orga0950412014-05-29 16:52:40 +0000101 SkRecorder translateCanvas(&translateRecord, W, H);
commit-bot@chromium.org0a98d872014-05-19 15:15:24 +0000102 SkMatrix translate;
103 translate.setTranslate(20, 20);
104 translateCanvas.setMatrix(translate);
105
halcanary96fcdcc2015-08-27 07:41:13 -0700106 SkRecordDraw(scaleRecord, &translateCanvas, nullptr, nullptr, 0, nullptr/*bbh*/, nullptr/*callback*/);
Mike Kleinc11530e2014-06-24 11:29:06 -0400107 REPORTER_ASSERT(r, 4 == translateRecord.count());
Mike Reed420a9ba2020-11-25 13:37:30 -0500108 assert_type<SkRecords::SetM44>(r, translateRecord, 0);
109 assert_type<SkRecords::Save> (r, translateRecord, 1);
110 assert_type<SkRecords::SetM44>(r, translateRecord, 2);
Mike Kleinc11530e2014-06-24 11:29:06 -0400111 assert_type<SkRecords::Restore> (r, translateRecord, 3);
commit-bot@chromium.org0a98d872014-05-19 15:15:24 +0000112
113 // When we look at translateRecord now, it should have its first +20,+20 translate,
114 // then a 2x,3x scale that's been concatted with that +20,+20 translate.
Mike Reed420a9ba2020-11-25 13:37:30 -0500115 const SkRecords::SetM44* setMatrix;
116 setMatrix = assert_type<SkRecords::SetM44>(r, translateRecord, 0);
117 REPORTER_ASSERT(r, setMatrix->matrix == SkM44(translate));
118
119 setMatrix = assert_type<SkRecords::SetM44>(r, translateRecord, 2);
120 SkMatrix expected = scale;
121 expected.postConcat(translate);
122 REPORTER_ASSERT(r, setMatrix->matrix == SkM44(expected));
commit-bot@chromium.org0a98d872014-05-19 15:15:24 +0000123}
mtkleina723b572014-08-15 11:49:49 -0700124
mtklein937c9c72014-09-02 15:19:48 -0700125// Like a==b, with a little slop recognizing that float equality can be weird.
126static bool sloppy_rect_eq(SkRect a, SkRect b) {
127 SkRect inset(a), outset(a);
128 inset.inset(1, 1);
129 outset.outset(1, 1);
130 return outset.contains(b) && !inset.contains(b);
131}
132
Mike Klein738b80d2018-05-04 13:51:11 -0400133// TODO This would be nice, but we can't get it right today.
134#if 0
mtklein40732b32015-10-24 07:45:47 -0700135DEF_TEST(RecordDraw_BasicBounds, r) {
mtkleina723b572014-08-15 11:49:49 -0700136 SkRecord record;
mtkleina723b572014-08-15 11:49:49 -0700137 SkRecorder recorder(&record, W, H);
138 recorder.save();
139 recorder.clipRect(SkRect::MakeWH(400, 500));
140 recorder.scale(2, 2);
141 recorder.drawRect(SkRect::MakeWH(320, 240), SkPaint());
142 recorder.restore();
143
mtklein40732b32015-10-24 07:45:47 -0700144 SkAutoTMalloc<SkRect> bounds(record.count());
145 SkRecordFillBounds(SkRect::MakeWH(SkIntToScalar(W), SkIntToScalar(H)), record, bounds);
mtkleina723b572014-08-15 11:49:49 -0700146
mtklein40732b32015-10-24 07:45:47 -0700147 for (int i = 0; i < record.count(); i++) {
148 REPORTER_ASSERT(r, sloppy_rect_eq(SkRect::MakeWH(400, 480), bounds[i]));
mtkleina723b572014-08-15 11:49:49 -0700149 }
150}
Mike Klein738b80d2018-05-04 13:51:11 -0400151#endif
mtklein00f30bd2014-09-02 12:03:31 -0700152
153// Base test to ensure start/stop range is respected
154DEF_TEST(RecordDraw_PartialStartStop, r) {
155 static const int kWidth = 10, kHeight = 10;
156
157 SkRect r1 = { 0, 0, kWidth, kHeight };
158 SkRect r2 = { 0, 0, kWidth, kHeight/2 };
159 SkRect r3 = { 0, 0, kWidth/2, kHeight };
160 SkPaint p;
161
162 SkRecord record;
163 SkRecorder recorder(&record, kWidth, kHeight);
164 recorder.drawRect(r1, p);
165 recorder.drawRect(r2, p);
166 recorder.drawRect(r3, p);
167
168 SkRecord rerecord;
169 SkRecorder canvas(&rerecord, kWidth, kHeight);
Mike Reed1a4140e2020-12-03 11:21:31 -0500170 SkRecordPartialDraw(record, &canvas, nullptr, 0, 1, 2, SkM44()); // replay just drawRect of r2
mtklein00f30bd2014-09-02 12:03:31 -0700171
reed2ff1fce2014-12-11 07:07:37 -0800172 REPORTER_ASSERT(r, 1 == count_instances_of_type<SkRecords::DrawRect>(rerecord));
173 int index = find_first_instances_of_type<SkRecords::DrawRect>(rerecord);
174 const SkRecords::DrawRect* drawRect = assert_type<SkRecords::DrawRect>(r, rerecord, index);
mtklein00f30bd2014-09-02 12:03:31 -0700175 REPORTER_ASSERT(r, drawRect->rect == r2);
176}
177
halcanary6950de62015-11-07 05:29:00 -0800178// A regression test for crbug.com/415468 and https://bug.skia.org/2957 .
mtklein8e393bf2014-10-01 12:48:58 -0700179//
180// This also now serves as a regression test for crbug.com/418417. We used to adjust the
181// bounds for the saveLayer, clip, and restore to be greater than the bounds of the picture.
182// (We were applying the saveLayer paint to the bounds after restore, which makes no sense.)
Mike Klein271a0302014-09-23 15:28:38 -0400183DEF_TEST(RecordDraw_SaveLayerAffectsClipBounds, r) {
184 SkRecord record;
185 SkRecorder recorder(&record, 50, 50);
186
187 // We draw a rectangle with a long drop shadow. We used to not update the clip
188 // bounds based on SaveLayer paints, so the drop shadow could be cut off.
189 SkPaint paint;
Michael Ludwig55edb502019-08-05 10:41:10 -0400190 paint.setImageFilter(SkImageFilters::DropShadow(20, 0, 0, 0, SK_ColorBLACK, nullptr));
Mike Klein271a0302014-09-23 15:28:38 -0400191
halcanary96fcdcc2015-08-27 07:41:13 -0700192 recorder.saveLayer(nullptr, &paint);
Mike Klein271a0302014-09-23 15:28:38 -0400193 recorder.clipRect(SkRect::MakeWH(20, 40));
194 recorder.drawRect(SkRect::MakeWH(20, 40), SkPaint());
195 recorder.restore();
196
mtklein8e393bf2014-10-01 12:48:58 -0700197 // Under the original bug, the right edge value of the drawRect would be 20 less than asserted
198 // here because we intersected it with a clip that had not been adjusted for the drop shadow.
199 //
200 // The second bug showed up as adjusting the picture bounds (0,0,50,50) by the drop shadow too.
201 // The saveLayer, clipRect, and restore bounds were incorrectly (0,0,70,50).
mtklein40732b32015-10-24 07:45:47 -0700202 SkAutoTMalloc<SkRect> bounds(record.count());
Mike Klein4c690b42020-02-26 10:30:29 -0600203 SkAutoTMalloc<SkBBoxHierarchy::Metadata> meta(record.count());
204 SkRecordFillBounds(SkRect::MakeWH(50, 50), record, bounds, meta);
mtklein40732b32015-10-24 07:45:47 -0700205 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[0], SkRect::MakeLTRB(0, 0, 50, 50)));
206 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[1], SkRect::MakeLTRB(0, 0, 50, 50)));
207 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[2], SkRect::MakeLTRB(0, 0, 40, 40)));
208 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[3], SkRect::MakeLTRB(0, 0, 50, 50)));
Mike Klein271a0302014-09-23 15:28:38 -0400209}
piotaixr65151752014-10-16 11:58:39 -0700210
Mike Klein4c690b42020-02-26 10:30:29 -0600211DEF_TEST(RecordDraw_Metadata, r) {
212 SkRecord record;
213 SkRecorder recorder(&record, 50, 50);
214
215 // Just doing some mildly interesting drawing, mostly grabbed from the unit test above.
216 SkPaint paint;
217 paint.setImageFilter(SkImageFilters::DropShadow(20, 0, 0, 0, SK_ColorBLACK, nullptr));
218
219 recorder.saveLayer(nullptr, &paint);
220 recorder.clipRect(SkRect::MakeWH(20, 40));
221 recorder.save();
222 recorder.translate(10, 10);
223 recorder.drawRect(SkRect::MakeWH(20, 40), SkPaint());
224 recorder.restore();
225 recorder.restore();
226
227 SkAutoTMalloc<SkRect> bounds(record.count());
228 SkAutoTMalloc<SkBBoxHierarchy::Metadata> meta(record.count());
229 SkRecordFillBounds(SkRect::MakeWH(50, 50), record, bounds, meta);
230
231 REPORTER_ASSERT(r, !meta[0].isDraw); // saveLayer (not a draw, but its restore will be)
232 REPORTER_ASSERT(r, !meta[1].isDraw); // clip
233 REPORTER_ASSERT(r, !meta[2].isDraw); // save
234 REPORTER_ASSERT(r, !meta[3].isDraw); // translate
235 REPORTER_ASSERT(r, meta[4].isDraw); // drawRect
236 REPORTER_ASSERT(r, !meta[5].isDraw); // restore (paired with save, not a draw)
237 REPORTER_ASSERT(r, meta[6].isDraw); // restore (paired with saveLayer, a draw)
238}
239
Mike Klein738b80d2018-05-04 13:51:11 -0400240// TODO This would be nice, but we can't get it right today.
241#if 0
robertphillips4d52afe2014-11-03 08:19:44 -0800242// When a saveLayer provides an explicit bound and has a complex paint (e.g., one that
243// affects transparent black), that bound should serve to shrink the area of the required
244// backing store.
245DEF_TEST(RecordDraw_SaveLayerBoundsAffectsClipBounds, r) {
246 SkRecord record;
247 SkRecorder recorder(&record, 50, 50);
248
249 SkPaint p;
reed374772b2016-10-05 17:33:02 -0700250 p.setBlendMode(SkBlendMode::kSrc);
robertphillips4d52afe2014-11-03 08:19:44 -0800251
mtklein40732b32015-10-24 07:45:47 -0700252 SkRect layerBounds = SkRect::MakeLTRB(10, 10, 40, 40);
253 recorder.saveLayer(&layerBounds, &p);
robertphillips4d52afe2014-11-03 08:19:44 -0800254 recorder.drawRect(SkRect::MakeLTRB(20, 20, 30, 30), SkPaint());
255 recorder.restore();
256
mtklein40732b32015-10-24 07:45:47 -0700257 SkAutoTMalloc<SkRect> bounds(record.count());
258 SkRecordFillBounds(SkRect::MakeWH(50, 50), record, bounds);
Cary Clarke041e312018-03-06 13:00:52 -0500259 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[0], SkRect::MakeLTRB(10, 10, 40, 40)));
260 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[1], SkRect::MakeLTRB(20, 20, 30, 30)));
261 REPORTER_ASSERT(r, sloppy_rect_eq(bounds[2], SkRect::MakeLTRB(10, 10, 40, 40)));
robertphillips4d52afe2014-11-03 08:19:44 -0800262}
Mike Klein738b80d2018-05-04 13:51:11 -0400263#endif
robertphillips4d52afe2014-11-03 08:19:44 -0800264
piotaixr65151752014-10-16 11:58:39 -0700265DEF_TEST(RecordDraw_drawImage, r){
266 class SkCanvasMock : public SkCanvas {
267 public:
mtkleinad8aa1d2014-11-11 18:52:02 -0800268 SkCanvasMock(int width, int height) : SkCanvas(width, height) {
piotaixr65151752014-10-16 11:58:39 -0700269 this->resetTestValues();
270 }
piotaixr65151752014-10-16 11:58:39 -0700271
piotaixr65151752014-10-16 11:58:39 -0700272 void resetTestValues() {
273 fDrawImageCalled = fDrawImageRectCalled = false;
274 }
275
276 bool fDrawImageCalled;
277 bool fDrawImageRectCalled;
piotaixr65151752014-10-16 11:58:39 -0700278 };
279
reede8f30622016-03-23 18:59:25 -0700280 auto surface(SkSurface::MakeRasterN32Premul(10, 10));
piotaixr65151752014-10-16 11:58:39 -0700281 surface->getCanvas()->clear(SK_ColorGREEN);
reed9ce9d672016-03-17 10:51:11 -0700282 sk_sp<SkImage> image(surface->makeImageSnapshot());
piotaixr65151752014-10-16 11:58:39 -0700283
284 SkCanvasMock canvas(10, 10);
piotaixr65151752014-10-16 11:58:39 -0700285}