use SkData::NewUninitialized

include/core/SkStream.h

Index: include/core/SkStream.h
diff --git a/include/core/SkStream.h b/include/core/SkStream.h
index 516b036a55e67162227374bf94350eb028ad7327..51ced24e56cf8a6c083ff88f7435e2d132e24909 100644
--- a/include/core/SkStream.h
+++ b/include/core/SkStream.h
@@ -56,6 +56,13 @@ public:
      */
     virtual size_t read(void* buffer, size_t size) = 0;
 
+    /**
+     *  Attempt to read size bytes into a SkData. If the read success, return the data,
+     *  else return NULL. Either way the stream's cursor may have been changed as a result
+     *  of calling read().
+     */
+    SkData* readIntoData(size_t size);

[bungeman-skia, 2014/09/12 14:50:27]

Eck, not another thing on SkStream...

It seems like this might be better (both in implementation and use) as

SkData* SkData::NewFromStream(SkStream* stream, size_t length = SIZE_MAX)

This gets this away from being yet another helper on SkStream. Since this isn't
virtual and always just calls public api and makes a copy of the stream data
anyway, it seems more clear that this is a SkData creator.

In the future this could even rely on a virtual 'SkData* refData()' which would
replace or augment getMemoryBase to not create a copy if the whole stream needed
to be duplicated into an SkData.

At the very least this would go into the readXXX non-virtual ghetto below.

[reed1, 2014/09/12 18:55:17]

Done.

+
     /** Skip size number of bytes.
      *  @return the actual number bytes that could be skipped.
      */