Ganesh: images upload to GPU performance fix (skip copying encoded data)

third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.h

Index: third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.h
diff --git a/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.h b/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.h
index 39d3c40b00713a5849d327db5aa0bc9b17454317..af997c521770c112aff9b3ed4c71f04bdd69f5a1 100644
--- a/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.h
+++ b/third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.h
@@ -27,6 +27,7 @@
 #define ImageFrameGenerator_h
 
 #include "SkBitmap.h"
+#include "SkData.h"

[scroggo_chromium, 2015/12/02 20:15:02]

Should this be forward-declared instead? (Blink style guide does not mention it,
but Chromium prefers it.)

 #include "SkSize.h"
 #include "SkTypes.h"
 #include "platform/PlatformExport.h"
@@ -75,8 +76,9 @@ public:
 
     void setData(PassRefPtr<SharedBuffer>, bool allDataReceived);
 
-    // Creates a new SharedBuffer containing the data received so far.
-    void copyData(RefPtr<SharedBuffer>*, bool* allDataReceived);
+    // Returns pointer to SkData. Caller needs to unref it, according to contract in
+    // SkImageGenerator::refEncodedData.

[scroggo_chromium, 2015/12/02 20:15:02]

Should this be called refEncodedData also, since it behaves similarly?

+    SkData* refSkData();
 
     SkISize getFullSize() const { return m_fullSize; }
 
@@ -104,7 +106,7 @@ private:
     bool decode(size_t index, ImageDecoder**, SkBitmap*);
 
     SkISize m_fullSize;
-    ThreadSafeDataTransport m_data;
+    RefPtr<ThreadSafeDataTransport> m_data;
     bool m_isMultiFrame;
     bool m_decodeFailedAndEmpty;
     Vector<bool> m_hasAlpha;