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[12], const SkColor colors[4]);
 
     /**
      * 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 {
         points[0] = fCtrlPoints[kTopP0_CubicCtrlPts];
@@ skipping 31 matching lines @@
     }
     
     const SkPoint* getControlPoints() const {
         return fCtrlPoints;
     }
     
     const SkColor* getColors() const {
         return fCornerColors;
     }
     
+    void setPoints(const SkPoint points[12]) {
+        memcpy(fCtrlPoints, points, kNumCtrlPts * sizeof(SkPoint));

[robertphillips, 2014/08/05 15:42:00]

extra line here ?

+
+    }
+    
+    void setColors(const SkColor colors[4]) {
+        memcpy(fCornerColors, colors, kNumColors * sizeof(SkColor));
+    }
+
+    void reset(const SkPoint points[12], const SkColor colors[4]) {
+        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 15:42:00]

Should the rest of this comment not be:

* Returns the number of bytes read (must be a multiple of 4) or
*             0 if there was not enough memory available

?

[dandov, 2014/08/05 17:34:05]

Done. Fixed function to follow the comment. 

+     * returns the number of bytes read
+     * 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