Web Animations CSS: Start running animations on the compositor

Source/core/animation/Player.cpp

Index: Source/core/animation/Player.cpp
diff --git a/Source/core/animation/Player.cpp b/Source/core/animation/Player.cpp
index 92c7cf4ca491e513fae3cdfbcca78171125c9071..ba19d3bc470ba7440ff02ddd280b83fe6d8e1988 100644
--- a/Source/core/animation/Player.cpp
+++ b/Source/core/animation/Player.cpp
@@ -32,17 +32,12 @@
 #include "config.h"
 #include "core/animation/Player.h"
 
+#include "core/animation/Animation.h"
 #include "core/animation/DocumentTimeline.h"
 #include "core/animation/TimedItem.h"
 
 namespace WebCore {
 
-namespace {
-
-double effectiveTime(double time) { return isNull(time) ? 0 : time; }
-
-} // namespace
-
 PassRefPtr<Player> Player::create(DocumentTimeline& timeline, TimedItem* content)
 {
     return adoptRef(new Player(timeline, content));
@@ -82,6 +77,30 @@ double Player::currentTimeBeforeDrift() const
     return (effectiveTime(m_timeline.currentTime()) - startTime()) * m_playbackRate;
 }
 
+bool Player::startCompositorAnimations()
+{
+    // FIXME: Support starting compositor animations that have a fixed
+    // start time.
+    ASSERT(!hasStartTime());
+    if (!m_content || !m_content->isAnimation())
+        return false;
+
+    return toAnimation(m_content.get())->startCompositorAnimations();
+}
+
+bool Player::isRunningCompositorAnimation()
+{
+    if (!m_content || !m_content->isAnimation())
+        return false;
+    return toAnimation(m_content.get())->isRunningCompositorAnimation();
+}
+
+void Player::cancelCompositorAnimations()
+{
+    if (isRunningCompositorAnimation())
+        toAnimation(m_content.get())->cancelCompositorAnimations();
+}
+
 double Player::pausedTimeDrift() const
 {
     ASSERT(pausedInternal());
@@ -139,7 +158,8 @@ void Player::setCurrentTime(double seekTime)
 
 void Player::pauseForTesting()
 {
-    ASSERT(!paused());
+    RELEASE_ASSERT(!isRunningCompositorAnimation());

[Steve Block, 2013/11/18 05:03:03]

Can you explain why you can assert this?

[dstockwell, 2013/11/18 06:11:20]

Done.

+    RELEASE_ASSERT(!paused());
     m_isPausedForTesting = true;
     setPausedImpl(true);
 }
@@ -155,9 +175,10 @@ void Player::setPausedImpl(bool newValue)
     if (pausedInternal() == newValue)
         return;
 
-    if (newValue)
+    if (newValue) {
+        cancelCompositorAnimations();
         m_pauseStartTime = currentTime();
-    else {
+    } else {
         m_timeDrift = pausedTimeDrift();
         m_pauseStartTime = nullValue();
     }