check for wide-open inside quickContains

src/gpu/GrClipStackClip.cpp

Index: src/gpu/GrClipStackClip.cpp
diff --git a/src/gpu/GrClipStackClip.cpp b/src/gpu/GrClipStackClip.cpp
index 695eda981fa379456995dd0ae1f2e22c678b38e0..7138f4b68a1d42bf6211b2a9f1dd36adf556a494 100644
--- a/src/gpu/GrClipStackClip.cpp
+++ b/src/gpu/GrClipStackClip.cpp
@@ -26,7 +26,7 @@ typedef GrReducedClip::ElementList ElementList;
 static const int kMaxAnalyticElements = 4;
 
 bool GrClipStackClip::quickContains(const SkRect& rect) const {
-    if (!fStack) {
+    if (!fStack || fStack->isWideOpen()) {

[bsalomon, 2016/08/22 19:47:56]

We don't need this call to isWideOpen() anymore, right?

[reed1, 2016/08/22 19:49:49]

Well, this is why my first version didn't put the check in clipstack: the cost
of making the rect itself (esp. since there are two int-to-float conversions)
was part of the slow-down. If I rely on the clipstack check, I still pay for the
rect.

         return true;
     }
     return fStack->quickContains(rect.makeOffset(SkIntToScalar(fOrigin.x()),
@@ -34,7 +34,7 @@ bool GrClipStackClip::quickContains(const SkRect& rect) const {
 }
 
 bool GrClipStackClip::quickContains(const SkRRect& rrect) const {
-    if (!fStack) {
+    if (!fStack || fStack->isWideOpen()) {

[bsalomon, 2016/08/22 19:47:56]

or this one.

         return true;
     }
     return fStack->quickContains(rrect.makeOffset(SkIntToScalar(fOrigin.fX),