Adding size parameter to read array functions

include/core/SkFlattenableBuffers.h

Index: include/core/SkFlattenableBuffers.h
diff --git a/include/core/SkFlattenableBuffers.h b/include/core/SkFlattenableBuffers.h
index 6e44550be561c6f267ade5f6655557a03d2edb3f..63badd729d51b09661b2a0fec0ae9c3ff1375786 100644
--- a/include/core/SkFlattenableBuffers.h
+++ b/include/core/SkFlattenableBuffers.h
@@ -99,11 +99,11 @@ public:
     virtual void readPath(SkPath* path) = 0;
 
     // binary data and arrays
-    virtual uint32_t readByteArray(void* value) = 0;
-    virtual uint32_t readColorArray(SkColor* colors) = 0;
-    virtual uint32_t readIntArray(int32_t* values) = 0;
-    virtual uint32_t readPointArray(SkPoint* points) = 0;
-    virtual uint32_t readScalarArray(SkScalar* values) = 0;
+    virtual uint32_t readByteArray(void* value, uint32_t size) = 0;

[reed1, 2013/10/23 21:10:10]

Lets add some dox here:

/**
 * The size parameter specifies the allocation size (in bytes? in elements?) of
the pointer parameter).
 */

[sugoi1, 2013/10/24 11:45:46]

Done.

+    virtual uint32_t readColorArray(SkColor* colors, uint32_t size) = 0;
+    virtual uint32_t readIntArray(int32_t* values, uint32_t size) = 0;
+    virtual uint32_t readPointArray(SkPoint* points, uint32_t size) = 0;
+    virtual uint32_t readScalarArray(SkScalar* values, uint32_t size) = 0;
 
     /** This helper peeks into the buffer and reports back the length of the next array in
      *  the buffer but does not change the state of the buffer.
@@ -127,7 +127,7 @@ public:
     SkData* readByteArrayAsData() {
         size_t len = this->getArrayCount();
         void* buffer = sk_malloc_throw(len);
-        (void)this->readByteArray(buffer);
+        (void)this->readByteArray(buffer, len);
         return SkData::NewFromMalloc(buffer, len);
     }