Check the condition for animation policy of bitmapImage on notifyFinished.

Source/core/rendering/RenderImage.cpp

Index: Source/core/rendering/RenderImage.cpp
diff --git a/Source/core/rendering/RenderImage.cpp b/Source/core/rendering/RenderImage.cpp
index 8e79299b78be056a39916d97ea90e4b8c768d33e..5fd28984643be781879505a408e9dcbdcb330451 100644
--- a/Source/core/rendering/RenderImage.cpp
+++ b/Source/core/rendering/RenderImage.cpp
@@ -203,11 +203,15 @@ void RenderImage::notifyFinished(Resource* newImage)
 
     invalidateBackgroundObscurationStatus();
 
-    if (newImage == m_imageResource->cachedImage()) {
+    ImageResource* image = m_imageResource->cachedImage();
+    if (newImage == image) {
         // tell any potential compositing layers
         // that the image is done and they can reference it directly.
         contentChanged(ImageChanged);
     }
+
+    if (image)
+        image->updateImageAnimationPolicy();

[fs, 2015/02/02 10:31:26]

This doesn't appear to cover all the cases compared to previously - the old way,
all RenderObject ImageResourceClients would "weigh in" to the outcome, but now
only RenderImage will (I'd expect at least RenderSVGImage to weigh in too, but
there could be other non-RenderImage-derived cases I'm missing...)

Also, if image != newImage, wouldn't you want this to be newImage? (I see the
issue with the type here.)

Additionally, if transforming "pull" to "push" it seems the value could be
looked up and passed directly, and hence avoid looping at all?

 }
 
 void RenderImage::paintReplaced(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)