Pre-crop filled rects to avoid scissor

include/gpu/GrDrawContext.h

Index: include/gpu/GrDrawContext.h
diff --git a/include/gpu/GrDrawContext.h b/include/gpu/GrDrawContext.h
index 96f42693b48ef06ce08400cc5d5864f8dfbfbf4a..8760dbe6683a864f71f92ff9bf471b2bc7d219a7 100644
--- a/include/gpu/GrDrawContext.h
+++ b/include/gpu/GrDrawContext.h
@@ -322,10 +322,13 @@ private:
                           const SkRRect& origOuter,
                           const SkRRect& origInner);
 
-    GrDrawBatch* getFillRectBatch(const GrPaint& paint,
-                                  const SkMatrix& viewMatrix,
-                                  const SkRect& rect,
-                                  bool* useHWAA);
+    // Returns false if the draw should be skipped entirely.
+    bool getFillRectBatch(const GrClip& clip,
+                          const GrPaint& paint,
+                          const SkMatrix& viewMatrix,
+                          const SkRect& rect,
+                          SkAutoTUnref<GrDrawBatch>* batch,

[csmartdalton, 2016/07/08 21:20:45]

I'm not sure I like this approach. Would it be better to return a batch, and set
a bool pointer-parameter that says whether to skip the draw?

[robertphillips, 2016/07/08 22:39:48]

If you update to ToT this call should be replaced with a drawFillRect entry
point. You should just be able to skip the draw in there.

[csmartdalton, 2016/07/11 03:17:56]

Done. Much better!

+                          bool* useHWAA);
 
     void internalDrawPath(const GrClip& clip,
                           const GrPaint& paint,