Stopped skipping tests in dm of SkPatch

include/core/SkPatch.h

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkPatch_DEFINED
 #define SkPatch_DEFINED
 
@@ skipping 59 matching lines @@
     };
     
     // Enum for corner colors also clockwise.
     enum CornerColors {
         kTopLeft_CornerColors = 0,
         kTopRight_CornerColors,
         kBottomRight_CornerColors,
         kBottomLeft_CornerColors
     };
     
+    enum {
+        kNumCtrlPts = 12,
+        kNumColors = 4,
+        kNumPtsCubic = 4
+    };
+    
     /**
      * Points are in the following order:
      *              (top curve)
      *                0 1 2 3
      * (left curve)  11     4   (right curve)
      *               10     5
      *                9 8 7 6
      *              (bottom curve)
      */
-    SkPatch(SkPoint points[12], SkColor colors[4]);
+    SkPatch() { }
+    SkPatch(const SkPoint points[kNumCtrlPts], const SkColor colors[kNumColors]);
 
     /**
      * Function that evaluates the coons patch interpolation.
      * data refers to the pointer of the PatchData struct in which the tessellation data is set.
      * lod refers the level of detail for each axis.
      */
     bool getVertexData(SkPatch::VertexData* data, int lodX, int lodY) const;
 
-    void getTopPoints(SkPoint points[4]) const {
+    void getTopPoints(SkPoint points[kNumPtsCubic]) const {
         points[0] = fCtrlPoints[kTopP0_CubicCtrlPts];
         points[1] = fCtrlPoints[kTopP1_CubicCtrlPts];
         points[2] = fCtrlPoints[kTopP2_CubicCtrlPts];
         points[3] = fCtrlPoints[kTopP3_CubicCtrlPts];
     }
     
-    void getBottomPoints(SkPoint points[4]) const {
+    void getBottomPoints(SkPoint points[kNumPtsCubic]) const {
         points[0] = fCtrlPoints[kBottomP0_CubicCtrlPts];
         points[1] = fCtrlPoints[kBottomP1_CubicCtrlPts];
         points[2] = fCtrlPoints[kBottomP2_CubicCtrlPts];
         points[3] = fCtrlPoints[kBottomP3_CubicCtrlPts];
     }
 
-    void getLeftPoints(SkPoint points[4]) const {
+    void getLeftPoints(SkPoint points[kNumPtsCubic]) const {
         points[0] = fCtrlPoints[kLeftP0_CubicCtrlPts];
         points[1] = fCtrlPoints[kLeftP1_CubicCtrlPts];
         points[2] = fCtrlPoints[kLeftP2_CubicCtrlPts];
         points[3] = fCtrlPoints[kLeftP3_CubicCtrlPts];
     }
 
-    void getRightPoints(SkPoint points[4]) const {
+    void getRightPoints(SkPoint points[kNumPtsCubic]) const {
         points[0] = fCtrlPoints[kRightP0_CubicCtrlPts];
         points[1] = fCtrlPoints[kRightP1_CubicCtrlPts];
         points[2] = fCtrlPoints[kRightP2_CubicCtrlPts];
         points[3] = fCtrlPoints[kRightP3_CubicCtrlPts];
     }
     

[robertphillips, 2014/08/05 12:44:45]

Not sure about this use of kNumPtsCubic. This could probably just be 4.

[dandov, 2014/08/05 14:27:11]

Done.

-    void getCornerPoints(SkPoint points[4]) const {
+    void getCornerPoints(SkPoint points[kNumPtsCubic]) const {
         points[0] = fCtrlPoints[kTopP0_CubicCtrlPts];
         points[1] = fCtrlPoints[kTopP3_CubicCtrlPts];
         points[2] = fCtrlPoints[kBottomP3_CubicCtrlPts];
         points[3] = fCtrlPoints[kBottomP0_CubicCtrlPts];
     }
     
     const SkPoint* getControlPoints() const {
         return fCtrlPoints;
     }
     
     const SkColor* getColors() const {
         return fCornerColors;
     }
     
+    void setPoints(const SkPoint points[kNumCtrlPts]) {
+        memcpy(fCtrlPoints, points, kNumCtrlPts * sizeof(SkPoint));
+
+    }
+    
+    void setColors(const SkColor colors[kNumColors]) {
+        memcpy(fCornerColors, colors, kNumColors * sizeof(SkColor));
+    }
+
+    void reset(const SkPoint points[kNumCtrlPts], const SkColor colors[kNumColors]) {
+        this->setPoints(points);
+        this->setColors(colors);
+    }
+
+    /**
+     *  Write the patch to the buffer, and return the number of bytes written.
+     *  If buffer is NULL, it still returns the number of bytes.
+     */
+    size_t writeToMemory(void* buffer) const;
+    
+    /**
+     * Initializes the patch from the buffer
+     *
+     * buffer Memory to read from
+     * length Amount of memory available in the buffer

[robertphillips, 2014/08/05 12:44:45]

Add a "Returns" before "number" ?

[dandov, 2014/08/05 14:27:11]

Done.

+     * number of bytes read (must be a multiple of 4) or
+     *         0 if there was not enough memory available
+     */
+    size_t readFromMemory(const void* buffer, size_t length);
+    
 private:
-    SkPoint fCtrlPoints[12];
-    SkColor fCornerColors[4];
+    SkPoint fCtrlPoints[kNumCtrlPts];
+    SkColor fCornerColors[kNumColors];
 };
 
 #endif