Drive SVG Animations via requestAnimationFrame

Source/core/page/PageAnimator.cpp

Index: Source/core/page/PageAnimator.cpp
diff --git a/Source/core/page/PageAnimator.cpp b/Source/core/page/PageAnimator.cpp
index 1998a8e4ad25dcdb70315e60f5b5aa47dc53c28a..8bca069bd0b727cada442ae55655b1278ca97414 100644
--- a/Source/core/page/PageAnimator.cpp
+++ b/Source/core/page/PageAnimator.cpp
@@ -12,6 +12,7 @@
 #include "core/page/Chrome.h"
 #include "core/page/ChromeClient.h"
 #include "core/page/Page.h"
+#include "core/svg/SVGDocumentExtensions.h"
 
 namespace WebCore {
 
@@ -30,6 +31,7 @@ void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
     for (RefPtr<LocalFrame> frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
         frame->view()->serviceScrollAnimations();
         DocumentAnimations::updateAnimationTimingForAnimationFrame(*frame->document(), monotonicAnimationStartTime);
+        SVGDocumentExtensions::serviceOnAnimationFrame(*frame->document(), monotonicAnimationStartTime);

[dstockwell, 2014/03/04 01:19:04]

We updated the naming in DocumentAnimations recently as for those outside the
animations systems it's not clear at all what 'service' is doing.

Dispatching of CSS Animation events then moved to the regular scripted animation
controller (see https://codereview.chromium.org/177203009/)

Are SVG Animation events ever fired synchronously?

[fs, 2014/03/04 10:28:19]

I noticed this, and for a while considered following suite. I seemed better to
keep the old naming until the new one made sense though (i.e. until we have both
components as CSS Animations do etc.) Hence, yet another thing I hope to get
aligned soon(TM).
Haven't looked into that much. YATTA ("Yet Another Thing To Align") I suppose
then.
AFAICR, no.

     }
 
     Vector<RefPtr<Document> > documents;