commit 80746c2bd2de5ce94c4cfd4429aeca47529df01a
author Florin Malita <fmalita@chromium.org> Sat Dec 30 12:39:11 2017 -0500
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Sat Dec 30 17:57:08 2017 +0000
tree 340ef62b9d6a180f40a73dc7ced7ed97517a4d41
parent 0f5726edf289460acd967a1a8a78327323878fca

[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>

experimental/sksg/SkSGInvalidationController.cpp

diff --git a/experimental/sksg/SkSGInvalidationController.cpp b/experimental/sksg/SkSGInvalidationController.cpp
index 114d724..4b133d8 100644
--- a/experimental/sksg/SkSGInvalidationController.cpp
+++ b/experimental/sksg/SkSGInvalidationController.cpp
@@ -17,8 +17,8 @@
 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);