Defer starting of animations until after compositing update

Source/core/page/PageAnimator.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/page/PageAnimator.h"
 
 #include "core/animation/DocumentAnimations.h"
 #include "core/dom/Document.h"
 #include "core/frame/Frame.h"
 #include "core/frame/FrameView.h"
 #include "core/page/Page.h"
 
 namespace WebCore {
 
 PageAnimator::PageAnimator(Page* page)
     : m_page(page)
     , m_servicingAnimations(false)
 {
 }
 
 void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
 {
     TemporaryChange<bool> servicing(m_servicingAnimations, true);
 
     for (RefPtr<Frame> frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
         frame->view()->serviceScrollAnimations();
-        DocumentAnimations::serviceOnAnimationFrame(*frame->document(), monotonicAnimationStartTime);
+        DocumentAnimations::updateAnimationTimingForAnimationFrame(*frame->document(), monotonicAnimationStartTime);
+        DocumentAnimations::dispatchAnimationEvents(*frame->document());

[esprehn, 2014/02/24 23:14:35]

This should have a FIXME that we should move the animation events logic into
ScriptedAnimationController.

Now that this is split we can trivially do that too, just move the call to
DocumentAnimations::dispatchAnimationEvents into
ScriptedAnimationController::serviceScriptedAnimations.

[dstockwell, 2014/02/25 00:52:03]

Done.

     }
 
     Vector<RefPtr<Document> > documents;
     for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext())
         documents.append(frame->document());
 
     for (size_t i = 0; i < documents.size(); ++i)
         documents[i]->serviceScriptedAnimations(monotonicAnimationStartTime);
 }
 
 }