Web Animations CSS: Start running animations on the compositor

Source/core/animation/Player.h

Index: Source/core/animation/Player.h
diff --git a/Source/core/animation/Player.h b/Source/core/animation/Player.h
index b4b4d81c1a9e7f2a5efb6657b73e4704984a95f0..17c68b85876852af26447f3878aa24cf1fe17d47 100644
--- a/Source/core/animation/Player.h
+++ b/Source/core/animation/Player.h
@@ -31,12 +31,14 @@
 #ifndef Player_h
 #define Player_h
 
+#include "CSSPropertyNames.h"

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

Superfluous

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

Done.

 #include "core/animation/TimedItem.h"
 #include "wtf/RefPtr.h"
 
 namespace WebCore {
 
 class DocumentTimeline;
+class Element;

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

Superfluous

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

Done.

 
 class Player FINAL : public RefCounted<Player> {
 
@@ -44,6 +46,8 @@ public:
     ~Player();
     static PassRefPtr<Player> create(DocumentTimeline&, TimedItem*);
 
+    static double effectiveTime(double time) { return isNull(time) ? 0 : time; }

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

Why is this now in the header? I don't see it used anywhere outside the cpp. If
it does need to be public, Player seems like an odd place for it to live. Maybe
Timing or TimedItem?

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

Unused, was missed in a refactor. Removed.

+
     // Returns whether this player is still current or in effect.
     // timeToEffectChange returns:
     //  infinity  - if this player is no longer in effect
@@ -75,11 +79,16 @@ public:
     // Reflects all pausing, including via pauseForTesting().
     bool pausedInternal() const { return !isNull(m_pauseStartTime); }
 
+    bool startCompositorAnimations();
+    void cancelCompositorAnimations();
+
 private:
     Player(DocumentTimeline&, TimedItem*);
     inline double pausedTimeDrift() const;
     inline double currentTimeBeforeDrift() const;
 
+    bool isRunningCompositorAnimation();
+
     void setPausedImpl(bool);
 
     double m_pauseStartTime;