adding rect batch unit test

src/gpu/GrRectBatch.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "GrRectBatch.h"
 
 #include "GrBatch.h"
 #include "GrBatchTarget.h"
+#include "GrBatchTest.h"
 #include "GrDefaultGeoProcFactory.h"
 #include "GrPrimitiveProcessor.h"
 #include "GrTemplates.h"
 
 /** We always use per-vertex colors so that rects can be batched across color changes. Sometimes we
     have explicit local coords and sometimes not. We *could* always provide explicit local coords
     and just duplicate the positions when the caller hasn't provided a local coord rect, but we
     haven't seen a use case which frequently switches between local rect and no local rect draws.
 
     The color param is used to determine whether the opaque hint can be set on the draw state.
@@ skipping 12 matching lines @@
                                                GrColorIsOpaque(color));
     } else {
         return GrDefaultGeoProcFactory::Create(flags, color, SkMatrix::I(), SkMatrix::I(),
                                                GrColorIsOpaque(color));
     }
 }
 
 class RectBatch : public GrBatch {
 public:
     struct Geometry {
-        GrColor fColor;
         SkMatrix fViewMatrix;
         SkRect fRect;
+        SkRect fLocalRect;
+        SkMatrix fLocalMatrix;
+        GrColor fColor;
         bool fHasLocalRect;
         bool fHasLocalMatrix;
-        SkRect fLocalRect;
-        SkMatrix fLocalMatrix;
     };
 
     static GrBatch* Create(const Geometry& geometry) {
         return SkNEW_ARGS(RectBatch, (geometry));
     }
 
     const char* name() const override { return "RectBatch"; }
 
     void getInvariantOutputColor(GrInitInvariantOutput* out) const override {
         // When this is called on a batch, there is only one geometry bundle
@@ skipping 181 matching lines @@
         geometry.fHasLocalMatrix = true;
         geometry.fLocalMatrix = *localMatrix;
     } else {
         geometry.fHasLocalMatrix = false;
     }
 
     return RectBatch::Create(geometry);
 }
 
 };
+
+///////////////////////////////////////////////////////////////////////////////////////////////////
+
+#ifdef GR_TEST_UTILS
+
+BATCH_TEST_DEFINE(RectBatch) {
+    GrColor color = GrRandomColor(random);
+

[robertphillips, 2015/05/07 16:33:21]

Don't bother initing viewMatrix and/or localMatrix here ?
Move both decls down closer to where they are used ?

+    SkMatrix viewMatrix = SkMatrix::I();
+    SkMatrix localMatrix = SkMatrix::I();
+    SkRect rect = GrTest::TestRect(random);
+    SkRect localRect;
+    bool hasLocalRect = random->nextBool();
+    bool hasLocalMatrix = random->nextBool();
+
+    if (hasLocalRect) {
+        viewMatrix = GrTest::TestMatrixInvertible(random);
+        localRect = GrTest::TestRect(random);
+    } else {
+        viewMatrix = GrTest::TestMatrix(random);
+    }
+
+    if (hasLocalMatrix) {
+        localMatrix = GrTest::TestMatrix(random);
+    }
+
+    return GrRectBatch::Create(color, viewMatrix, rect,
+                               hasLocalRect ? &localRect : NULL,
+                               hasLocalMatrix ? &localMatrix : NULL);
+}
+
+#endif