Switch SkPDFStream's internal storage from SkStream to SkData

src/pdf/SkPDFStream.h

Index: src/pdf/SkPDFStream.h
diff --git a/src/pdf/SkPDFStream.h b/src/pdf/SkPDFStream.h
index 6371bc187d14d9c90ed463d7734cfa67776f4d2e..2b58d489f3f6197229de4a8e8116154b33409ea9 100644
--- a/src/pdf/SkPDFStream.h
+++ b/src/pdf/SkPDFStream.h
@@ -74,9 +74,9 @@ protected:
     void setData(SkData* data);
     void setData(SkStream* stream);
 
-    SkStream* getData() {
-        return fData.get();
-    }
+    size_t dataSize();
+
+    SkData* getData() { return fData.get(); }
 
     void setState(State state) {
         fState = state;
@@ -90,8 +90,9 @@ private:
     // Indicates what form (or if) the stream has been requested.
     State fState;
 
-    // TODO(vandebo): Use SkData (after removing deprecated constructor).
-    SkAutoTUnref<SkStream> fData;
+    SkMutex fMutex;

[mtklein, 2014/06/26 13:21:59]

It's nice to note which fields this guards.  // Guards all fields  is fine if
that's true.

[hal.canary, 2014/06/26 18:42:02]

Done.

+
+    SkAutoTUnref<SkData> fData;
     SkAutoTUnref<SkPDFStream> fSubstitute;
 
     typedef SkPDFDict INHERITED;