Adding release method to SkWriter32 and using it to not copy the ops data from SkPictureRecord to S…

include/core/SkWriter32.h

Index: include/core/SkWriter32.h
diff --git a/include/core/SkWriter32.h b/include/core/SkWriter32.h
index df1275f7b656829ad132f7b481508e42b53d99e7..ff32d381880de120dd2fe7bff532f5c799c39017 100644
--- a/include/core/SkWriter32.h
+++ b/include/core/SkWriter32.h
@@ -21,6 +21,8 @@
 #include "SkTDArray.h"
 #include "SkTypes.h"
 
+class SkData;
+
 class SkWriter32 : SkNoncopyable {
 public:
     /**
@@ -229,6 +231,12 @@ public:
         return stream->read(this->reservePad(length), length);
     }
 
+    /**
+     *  Release the internal buffer to the caller in an SkData.
+     *  The caller must call unref() when it is finished using the data.
+     */
+    SkData* release();

[reed1, 2014/02/07 16:06:49]

Modest vote for a more explicit name :

e.g.

releaseAsData() ?
detachAsData() ?  // seems like we use detach for this idea in other classes,
but maybe we use both

Also we should well-document that this also resets the state of the writer.  

[iancottrell, 2014/02/07 17:01:47]

Done.

+
 private:
     void growToAtLeast(size_t size);