Drawing an animated image to a canvas now behave as expected

Source/core/html/canvas/CanvasRenderingContext2D.cpp

Index: Source/core/html/canvas/CanvasRenderingContext2D.cpp
diff --git a/Source/core/html/canvas/CanvasRenderingContext2D.cpp b/Source/core/html/canvas/CanvasRenderingContext2D.cpp
index 829cdb8fb66916bdd22a5e9a786b74057c50b9d2..38958118a8d33c78ce1e8e279d2ef2fa865ba29b 100644
--- a/Source/core/html/canvas/CanvasRenderingContext2D.cpp
+++ b/Source/core/html/canvas/CanvasRenderingContext2D.cpp
@@ -61,6 +61,7 @@
 #include "core/rendering/RenderTheme.h"
 #include "platform/fonts/FontCache.h"
 #include "platform/geometry/FloatQuad.h"
+#include "platform/graphics/BitmapImage.h"
 #include "platform/graphics/DrawLooperBuilder.h"
 #include "platform/graphics/GraphicsContextStateSaver.h"
 #include "platform/text/TextRun.h"
@@ -1508,6 +1509,8 @@ void CanvasRenderingContext2D::drawImageInternal(CanvasImageSource* imageSource,
     if (srcRect.isEmpty())
         return;
 
+    image = image->imageForDefaultFrame();

[Justin Novosad, 2014/07/18 15:22:32]

Wait a minute.. according to the spec, the behavior of fetching the default
frame is specific to image elements. No need to call it for all types of image
sources. This call should go in HTMLImageElement::getSourceImageForCanvas.

[Rémi Piotaix, 2014/07/18 16:04:35]

Done.

+
     FloatRect dirtyRect = clipBounds;
     if (imageSource->isVideoElement()) {
         drawVideo(static_cast<HTMLVideoElement*>(imageSource), srcRect, dstRect);