Animation: Cancel same-property animations on compositor

Source/core/animation/AnimationPlayer.h

Index: Source/core/animation/AnimationPlayer.h
diff --git a/Source/core/animation/AnimationPlayer.h b/Source/core/animation/AnimationPlayer.h
index aa27c81ee81272afefce2f4e96d983650e4ccec0..d0563a0fd82047826e5b95123afb521a820b3dbb 100644
--- a/Source/core/animation/AnimationPlayer.h
+++ b/Source/core/animation/AnimationPlayer.h
@@ -33,6 +33,7 @@
 
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseProperty.h"
+#include "core/CSSPropertyNames.h"
 #include "core/animation/AnimationNode.h"
 #include "core/dom/ActiveDOMObject.h"
 #include "core/dom/DOMException.h"
@@ -140,11 +141,13 @@ public:
     bool canStartAnimationOnCompositor();
     bool maybeStartAnimationOnCompositor();
     void cancelAnimationOnCompositor();
+    void cancelAffectedAnimationsOnCompositor();
     bool hasActiveAnimationsOnCompositor();
     void setCompositorPending(bool sourceChanged = false);
     void notifyCompositorStartTime(double timelineTime);
     void notifyStartTime(double timelineTime);
 
+    bool affects(CSSPropertyID) const;

[dstockwell, 2015/01/28 05:44:05]

This should have an element for context (conceptually Players may target more
than one element):
bool affects(Element&, CSSPropertyID);

[loyso (OOO), 2015/01/28 06:33:31]

Is that possible with current codebase? AnimationPlayer contains just one
AnimationNode* m_content. Should we simply support that concept in a class API?

[dstockwell, 2015/01/29 01:14:07]

No it is not possible currently, but I would like to avoid having to plumb this
though later. If this function does not take Element* then the functions in
CompositorAnimations should only require ActiveAnimations* instead of Element.

 
     void preCommit(int compositorGroup, bool startOnCompositor);
     void postCommit(double timelineTime);