Add new skpinfo tool

include/core/SkPicture.h

Index: include/core/SkPicture.h
===================================================================
--- include/core/SkPicture.h	(revision 13626)
+++ include/core/SkPicture.h	(working copy)
@@ -217,6 +217,13 @@
     void abortPlayback();
 #endif
 
+    // Return true if the SkStream represents a serialized picture, and fills out
+    // SkPictInfo. After this function returns, the SkStream is not rewound; it
+    // will be ready to be parsed to create an SkPicturePlayback.

[reed1, 2014/03/03 15:18:24]

This part about "ready to be parsed..." is confusing. Is this meant to be a
private API, but exposed here for the tool?
I wonder if we can make that part clearer.

[robertphillips, 2014/03/03 15:29:29]

Done. Hopefully clearer.

+    // If false is returned, SkPictInfo is unmodified.
+    static bool StreamIsSKP(SkStream*, SkPictInfo*);
+    static bool BufferIsSKP(SkReadBuffer&, SkPictInfo*);
+
 protected:
     // V2 : adds SkPixelRef's generation ID.
     // V3 : PictInfo tag at beginning, and EOF tag at the end
@@ -262,13 +269,6 @@
     // For testing. Derived classes may instantiate an alternate
     // SkBBoxHierarchy implementation
     virtual SkBBoxHierarchy* createBBoxHierarchy() const;
-
-    // Return true if the SkStream represents a serialized picture, and fills out
-    // SkPictInfo. After this function returns, the SkStream is not rewound; it
-    // will be ready to be parsed to create an SkPicturePlayback.
-    // If false is returned, SkPictInfo is unmodified.
-    static bool StreamIsSKP(SkStream*, SkPictInfo*);
-    static bool BufferIsSKP(SkReadBuffer&, SkPictInfo*);
 private:
     void createHeader(void* header) const;