Fix ImageDecoder::frameIsCompleteAtIndex - fully received instead of decoded.

third_party/WebKit/Source/platform/graphics/test/MockImageDecoder.h

Index: third_party/WebKit/Source/platform/graphics/test/MockImageDecoder.h
diff --git a/third_party/WebKit/Source/platform/graphics/test/MockImageDecoder.h b/third_party/WebKit/Source/platform/graphics/test/MockImageDecoder.h
index 05fff2ecdc2786020b8829daa16fd3241bf81bc8..69a8aaf984e8660554b4cf33b9a1ebbf6fe0a9c4 100644
--- a/third_party/WebKit/Source/platform/graphics/test/MockImageDecoder.h
+++ b/third_party/WebKit/Source/platform/graphics/test/MockImageDecoder.h
@@ -92,9 +92,13 @@ public:
         return m_client->repetitionCount();
     }
 
-    bool frameIsCompleteAtIndex(size_t) const override
+    bool frameIsFullyReceivedAtIndex(size_t index) const override
     {
-        return m_client->status() == ImageFrame::FrameComplete;
+        // All but the last frame are fully received.
+        // For the last frame use base class implementation.
+        if (index < const_cast<MockImageDecoder*>(this)->frameCount() - 1)

[Peter Kasting, 2016/05/25 01:05:23]

Casting away const is really unpleasant.  frameCount() could change a variety of
things about this object.

Can we implement this without doing that somehow?  Maybe always call through to
the base class?  Maybe keep enough data in this class to answer the query
without calling frameCount()?

[aleksandar.stojiljkovic, 2016/05/30 16:12:19]

Done.

+            return true;
+        return ImageDecoder::frameIsFullyReceivedAtIndex(index);
     }
 
     float frameDurationAtIndex(size_t) const override