In PageAnimator iterate through collected documents instead of frames

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/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
@@ skipping 10 matching lines @@
     , m_servicingAnimations(false)
     , m_updatingLayoutAndStyleForPainting(false)
 {
 }
 
 void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
 {
     m_animationFramePending = false;
     TemporaryChange<bool> servicing(m_servicingAnimations, true);
 
-    for (RefPtr<Frame> frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
-        if (frame->isLocalFrame()) {
-            RefPtr<LocalFrame> localFrame = toLocalFrame(frame.get());
-            localFrame->view()->serviceScrollAnimations(monotonicAnimationStartTime);
-
-            DocumentAnimations::updateAnimationTimingForAnimationFrame(*localFrame->document(), monotonicAnimationStartTime);
-            SVGDocumentExtensions::serviceOnAnimationFrame(*localFrame->document(), monotonicAnimationStartTime);
-        }
-    }
-
     WillBeHeapVector<RefPtrWillBeMember<Document> > documents;
     for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
         if (frame->isLocalFrame())
             documents.append(toLocalFrame(frame)->document());
     }
 
+    for (size_t i = 0; i < documents.size(); ++i) {
+        if (documents[i]->frame())
+            documents[i]->view()->serviceScrollAnimations(monotonicAnimationStartTime);
+    }
+
+    for (size_t i = 0; i < documents.size(); ++i) {
+        DocumentAnimations::updateAnimationTimingForAnimationFrame(*documents[i], monotonicAnimationStartTime);
+        SVGDocumentExtensions::serviceOnAnimationFrame(*documents[i], monotonicAnimationStartTime);
+    }
+
     for (size_t i = 0; i < documents.size(); ++i)
         documents[i]->serviceScriptedAnimations(monotonicAnimationStartTime);
 }
 
 void PageAnimator::scheduleVisualUpdate()
 {
     // FIXME: also include m_animationFramePending here. It is currently not there due to crbug.com/353756.
     if (m_servicingAnimations || m_updatingLayoutAndStyleForPainting)
         return;
     m_page->chrome().scheduleAnimation();
@@ skipping 12 matching lines @@
     // setFrameRect(). This will be a quick operation for most frames, but the
     // NativeWindowWidgets will update a proper clipping region.
     view->setFrameRect(view->frameRect());
 
     // setFrameRect may have the side-effect of causing existing page layout to
     // be invalidated, so layout needs to be called last.
     view->updateLayoutAndStyleForPainting();
 }
 
 }