add runtime option to provide data->imagegenerator factory

include/core/SkImageGenerator.h

Index: include/core/SkImageGenerator.h
diff --git a/include/core/SkImageGenerator.h b/include/core/SkImageGenerator.h
index a398697e6ba4eaa1f521bbf6f2107f1af4876f82..46001c85370755ef69e28f2cc9f842268815ddcf 100644
--- a/include/core/SkImageGenerator.h
+++ b/include/core/SkImageGenerator.h
@@ -199,7 +199,7 @@ public:
      *  this returns a new ImageGenerator for it. Otherwise this returns NULL. Either way
      *  the caller is still responsible for managing their ownership of the data.
      */
-    static SkImageGenerator* NewFromData(SkData*);
+    static SkImageGenerator* NewFromEncoded(SkData*);
 
 protected:
     SkImageGenerator(const SkImageInfo& info) : fInfo(info) {}
@@ -220,6 +220,8 @@ protected:
 
 private:
     const SkImageInfo fInfo;
+
+    static SkImageGenerator* NewFromEncodedImpl(SkData*);

[scroggo, 2015/07/09 19:01:15]

Maybe add a comment that the implementation does not need to check for NULL?

[reed1, 2015/07/09 19:05:45]

Will do

 };
 
 #endif  // SkImageGenerator_DEFINED