Stopped skipping tests in dm of SkPatch

gm/patch.cpp

Index: gm/patch.cpp
diff --git a/gm/patch.cpp b/gm/patch.cpp
index 5658ef6a889d9d65788a708a7cff44e57259b9d6..3e3bf7a1b57fcc0c922a46e9758f215c8cd37ba9 100644
--- a/gm/patch.cpp
+++ b/gm/patch.cpp
@@ -62,7 +62,7 @@ static void draw_control_points(SkCanvas* canvas, SkPatch& patch, SkPaint& paint
 static void draw_random_patch(SkPoint points[12], SkColor colors[4], SkCanvas* canvas,
                               SkPaint& paint, SkRandom* rnd) {
     SkPoint ptsCpy[12];

[mtklein, 2014/08/04 20:40:47]

Seems like we still have many 12's and 4's.  It's pretty hard to understand code
that says kNumCtrlPts some places and 12 others.

Personally I don't mind which way it ends up --- all 12's or all kNumCtrlPts ---
but let's make it consistent everywhere in the codebase.

[dandov, 2014/08/04 21:47:54]

I think I fixed all of the 12's and 4's that are related with the number of
points or colors in a patch. 

-    memcpy(ptsCpy, points, 12 * sizeof(SkPoint));
+    memcpy(ptsCpy, points, SkPatch::kNumCtrlPts * sizeof(SkPoint));
     for (int i = 0; i < 5; i++) {
         int index = rnd->nextRangeU(0, 11);
         SkScalar dx = rnd->nextRangeScalar(-50, 50), dy = rnd->nextRangeScalar(-50, 50);
@@ -94,7 +94,7 @@ protected:
     }
 
     virtual uint32_t onGetFlags() const SK_OVERRIDE {
-        return kSkipTiled_Flag | kSkipPipe_Flag | kSkipPicture_Flag;
+        return kSkipTiled_Flag;
     }
     
     virtual void onDraw(SkCanvas* canvas) SK_OVERRIDE {