diff options
| author |  Florin Malita <fmalita@chromium.org> | 2017-12-30 12:39:11 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-12-30 17:57:08 +0000 |
| commit | 80746c2bd2de5ce94c4cfd4429aeca47529df01a (patch) | |
| tree | 340ef62b9d6a180f40a73dc7ced7ed97517a4d41 /experimental | |
| parent | 0f5726edf289460acd967a1a8a78327323878fca (diff) | |
[sksg] Fix inval rect mapping
Return false from mapRect() doesn't mean the op failed.
TBR=
Change-Id: I0582fde3efaa792010f27e3684cfe9c4332e29dc
Reviewed-on: https://skia-review.googlesource.com/90021
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Diffstat (limited to 'experimental')
| -rw-r--r-- | experimental/sksg/SkSGInvalidationController.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/experimental/sksg/SkSGInvalidationController.cpp b/experimental/sksg/SkSGInvalidationController.cpp index 114d724d1c..4b133d871b 100644 --- a/experimental/sksg/SkSGInvalidationController.cpp +++ b/experimental/sksg/SkSGInvalidationController.cpp @@ -17,8 +17,8 @@ InvalidationController::InvalidationController() {} void InvalidationController::inval(const SkRect& r, const SkMatrix& ctm) { SkTCopyOnFirstWrite<SkRect> rect(r); - if (!ctm.isIdentity() && !ctm.mapRect(rect.writable())) { - *rect.writable() = SkRect::MakeLTRB(SK_ScalarMin, SK_ScalarMin, SK_ScalarMax, SK_ScalarMax); + if (!ctm.isIdentity()) { + ctm.mapRect(rect.writable()); } fRects.push(*rect); |