Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3151)

Unified Diff: sky/engine/core/page/PageAnimator.cpp

Issue 1020053002: Remove pauseAnimationsForTesting (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/core/animation/AnimationTimelineTest.cpp ('k') | sky/engine/public/web/WebDocument.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/page/PageAnimator.cpp
diff --git a/sky/engine/core/page/PageAnimator.cpp b/sky/engine/core/page/PageAnimator.cpp
index bdc62a94197a343691da44937d981d520ede1b15..7a147848e963fa9bfbb507e83838734e00d18baa 100644
--- a/sky/engine/core/page/PageAnimator.cpp
+++ b/sky/engine/core/page/PageAnimator.cpp
@@ -26,17 +26,10 @@ void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
{
TemporaryChange<bool> servicing(m_servicingAnimations, true);
- Vector<RefPtr<Document> > documents;
- documents.append(m_page->mainFrame()->document());
+ RefPtr<Document> document = m_page->mainFrame()->document();
- WTF_LOG(ScriptedAnimationController, "PageAnimator::serviceScriptedAnimations: #documents = %d",
- static_cast<int>(documents.size()));
-
- for (size_t i = 0; i < documents.size(); ++i)
- DocumentAnimations::updateAnimationTimingForAnimationFrame(*documents[i], monotonicAnimationStartTime);
-
- for (size_t i = 0; i < documents.size(); ++i)
- documents[i]->serviceScriptedAnimations(monotonicAnimationStartTime);
+ DocumentAnimations::updateAnimationTimingForAnimationFrame(*document, monotonicAnimationStartTime);
+ document->serviceScriptedAnimations(monotonicAnimationStartTime);
}
void PageAnimator::scheduleVisualUpdate()
« no previous file with comments | « sky/engine/core/animation/AnimationTimelineTest.cpp ('k') | sky/engine/public/web/WebDocument.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698