When filling nested rect path check for empty inner and empty outer rects

src/gpu/batches/GrRectBatchFactory.cpp

Index: src/gpu/batches/GrRectBatchFactory.cpp
diff --git a/src/gpu/batches/GrRectBatchFactory.cpp b/src/gpu/batches/GrRectBatchFactory.cpp
index f144d1b5325ec40a0f4fe25bc9dd2f1ccdc1ce87..c64a3192009e71c56e574b081fce2a121188fddd 100644
--- a/src/gpu/batches/GrRectBatchFactory.cpp
+++ b/src/gpu/batches/GrRectBatchFactory.cpp
@@ -22,9 +22,14 @@ GrDrawBatch* CreateAAFillNestedRects(GrColor color,
     SkRect devOutside, devInside;
     viewMatrix.mapRect(&devOutside, rects[0]);
     viewMatrix.mapRect(&devInside, rects[1]);
-
-    return GrAAStrokeRectBatch::Create(color, viewMatrix, devOutside, devOutside, devInside, true,
-                                       devInside.isEmpty());
+    if (devInside.isEmpty()) {
+        if (devOutside.isEmpty()) {
+            return nullptr;
+        }

[robertphillips, 2016/05/02 19:15:22]

Here we pass down the viewMatrix & the untransformed rect

[bsalomon, 2016/05/02 19:45:28]

you're right, this is supposed to be the dev rect

+        return GrAAFillRectBatch::Create(color, viewMatrix, rects[0], devOutside);
+    }
+

[robertphillips, 2016/05/02 19:15:22]

but here we pass down the viewMatrix and the two transformed rects ?

+    return GrAAStrokeRectBatch::CreateFillBetweenRects(color, viewMatrix, devOutside, devInside);
 }
 
 };