High CPU and increased memory usage fix for high-res (GIF, WEBP...) animations.

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

Index: third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.h
diff --git a/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.h b/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.h
index 74ba6cabbe88bb061a142cb73f3539596b62f178..aef00b8a71e03ffdf80e60ac9c6c96e821eab4bb 100644
--- a/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.h
+++ b/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.h
@@ -48,9 +48,14 @@ class PLATFORM_EXPORT DecodingImageGenerator final : public SkImageGenerator {
     USING_FAST_MALLOC(DecodingImageGenerator);
     WTF_MAKE_NONCOPYABLE(DecodingImageGenerator);
 public:
+    // Make kNeedNeedNewImageUniqueID accessible.

[Peter Kasting, 2016/05/04 03:01:07]

Extra "need".  Also, you should qualify this with SkImageGenerator:: to be clear
to readers.

But frankly, I think better than this would be to make the value public on
SkImageGenerator.  If callers of this class are expected to use unique IDs which
are actually the ones used by SkImageGenerator it seems reasonable to expose
this magic value publicly there.

[aleksandar.stojiljkovic, 2016/05/04 20:56:29]

Done.

[Peter Kasting, 2016/05/07 01:58:29]

Any comment on the idea of exposing SkImageGenerator::kNeedNewImageUniqueID
publicly directly?

[aleksandar.stojiljkovic, 2016/05/07 19:50:52]

I think SkImageGenerator should stay as it is now -
SkImageGenerator::kNeedNewImageUniqueID is used only in protected method
(constructor is protected) and this const parameter used only in constructor
should be protected, too. It is client choice here to use it outside protected
class code so client needs to deal with this.

+    enum {
+        NeedNewImageUniqueID = kNeedNewImageUniqueID

[Peter Kasting, 2016/05/04 03:01:07]

This should still be named with a leading k.  Use a qualifier to disambiguate
the two?

[aleksandar.stojiljkovic, 2016/05/04 20:56:29]

Done.

+    };
+
     static SkImageGenerator* create(SkData*);
 
-    DecodingImageGenerator(PassRefPtr<ImageFrameGenerator>, const SkImageInfo&, PassRefPtr<SegmentReader>, bool allDataReceived, size_t index);
+    DecodingImageGenerator(PassRefPtr<ImageFrameGenerator>, const SkImageInfo&, PassRefPtr<SegmentReader>, bool allDataReceived, size_t index, uint32_t uniqueID = NeedNewImageUniqueID);
     ~DecodingImageGenerator() override;
 
     void setCanYUVDecode(bool yes) { m_canYUVDecode = yes; }