commit 02be3c1c3f53858263df82b1db466b5bb7059ab0
author Mike Reed <reed@google.com> Thu Mar 23 12:34:15 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Mar 23 17:26:34 2017 +0000
tree c06b8251806559c256d6643079b912639befd594
parent 78075804c1e43ca32f3d8eb75d2a8b6c91e74b9b

add test and fix for canvas::isClipEmpty

BUG=skia:

Change-Id: I4e57e0da7111e861fbae1b88574bc1803442243c
Reviewed-on: https://skia-review.googlesource.com/10046
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Derek Sollenberger <djsollen@google.com>

src/core/SkCanvas.cpp

diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp
index 45acc53..0301804 100644
--- a/src/core/SkCanvas.cpp
+++ b/src/core/SkCanvas.cpp
@@ -1610,9 +1610,14 @@
 ///////////////////////////////////////////////////////////////////////////////
 
 bool SkCanvas::isClipEmpty() const {
+    return fMCRec->fRasterClip.isEmpty();
+
+    // TODO: should we only use the conservative answer in a recording canvas?
+#if 0
     SkBaseDevice* dev = this->getTopDevice();
     // if no device we return true
     return !dev || dev->onGetClipType() == SkBaseDevice::kEmpty_ClipType;
+#endif
 }
 
 bool SkCanvas::isClipRect() const {