GIF Animations "clear from cache related glitch" fix

third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h

Index: third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h
diff --git a/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h b/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h
index 3ce2da9d3bbc71324c81ccc2e0559ece8a61b519..bf817f80be8d93efe2fd4114e7e6bda094219dc4 100644
--- a/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h
+++ b/third_party/WebKit/Source/platform/image-decoders/gif/GIFImageDecoder.h
@@ -80,6 +80,9 @@ private:
     // the previous frame's disposal method. Returns true on success. On
     // failure, this will mark the image as failed.
     bool initFrameBuffer(size_t frameIndex);
+    // If disposal method is DisposeOverwritePrevious, keep two frames: current

[Peter Kasting, 2016/01/14 00:17:29]

Nit: Blank line above this

[aleksandar.stojiljkovic, 2016/01/14 11:32:53]

Done.

+    // and it's required previous frame.

[Peter Kasting, 2016/01/14 00:17:29]

This comment is technically a bit of a layering violation, because this
describes how callers are using this rather than how the function actually
works.  I would do this:

  // Like clearCacheExceptFrame(), but preserves two frames instead of one.
  size_t clearCacheExceptFrames(size_t frame1, size_t frame2);

[aleksandar.stojiljkovic, 2016/01/14 11:32:53]

Done.

+    size_t clearCacheExceptTwoFrames(size_t dontClearFrame1, size_t dontClearFrame2);
 
     bool m_currentBufferSawAlpha;
     mutable int m_repetitionCount;