GLProgramsTest 3.0

src/gpu/GrOvalRenderer.cpp

Index: src/gpu/GrOvalRenderer.cpp
diff --git a/src/gpu/GrOvalRenderer.cpp b/src/gpu/GrOvalRenderer.cpp
index ebac42fcac8744c36f88d14ea5c05daf662bf68a..a4b986fa7639956fb7caf5257220edb6b276eb65 100644
--- a/src/gpu/GrOvalRenderer.cpp
+++ b/src/gpu/GrOvalRenderer.cpp
@@ -2094,7 +2094,7 @@ BATCH_TEST_DEFINE(CircleBatch) {
     SkMatrix viewMatrix = GrTest::TestMatrix(random);
     GrColor color = GrRandomColor(random);
     bool useCoverageAA = random->nextBool();

[robertphillips, 2015/05/13 13:47:22]

Why not TestRectSimple?

-    SkRect circle = GrTest::TestRect(random);
+    SkRect circle = SkRect::MakeWH(100.f, 100.f);
     return create_circle_batch(color, viewMatrix, useCoverageAA, circle,
                                GrTest::TestStrokeRec(random));
 }
@@ -2102,9 +2102,8 @@ BATCH_TEST_DEFINE(CircleBatch) {
 BATCH_TEST_DEFINE(EllipseBatch) {
     SkMatrix viewMatrix = GrTest::TestMatrixRectStaysRect(random);
     GrColor color = GrRandomColor(random);
-    bool useCoverageAA = random->nextBool();
-    SkRect ellipse = GrTest::TestRect(random);
-    return create_ellipse_batch(color, viewMatrix, useCoverageAA, ellipse,
+    SkRect ellipse = GrTest::TestRectSimple(random);
+    return create_ellipse_batch(color, viewMatrix, true, ellipse,
                                 GrTest::TestStrokeRec(random));
 }
 
@@ -2112,7 +2111,7 @@ BATCH_TEST_DEFINE(DIEllipseBatch) {
     SkMatrix viewMatrix = GrTest::TestMatrix(random);
     GrColor color = GrRandomColor(random);
     bool useCoverageAA = random->nextBool();
-    SkRect ellipse = GrTest::TestRect(random);
+    SkRect ellipse = GrTest::TestRectSimple(random);
     return create_diellipse_batch(color, viewMatrix, useCoverageAA, ellipse,
                                   GrTest::TestStrokeRec(random));
 }