Compositor animations for Independent CSS Transform Properties

Source/core/animation/DocumentAnimations.cpp

Index: Source/core/animation/DocumentAnimations.cpp
diff --git a/Source/core/animation/DocumentAnimations.cpp b/Source/core/animation/DocumentAnimations.cpp
index f262f56232118d9b1e0eff6f7412c12457c0726f..3534441c36ce5641dea22c57e835050baa2b5c7d 100644
--- a/Source/core/animation/DocumentAnimations.cpp
+++ b/Source/core/animation/DocumentAnimations.cpp
@@ -71,7 +71,10 @@ void DocumentAnimations::updateAnimationTimingForGetComputedStyle(Node& node, CS
     const Element& element = toElement(node);
     if (const ComputedStyle* style = element.computedStyle()) {
         if ((property == CSSPropertyOpacity && style->isRunningOpacityAnimationOnCompositor())
+            || (property == CSSPropertyRotate && style->isRunningTransformAnimationOnCompositor())

[Eric Willigers, 2015/06/30 03:18:23]

Perhaps introduce a bool representing property == CSSPropertyRotate || property
== ... .

Then style->isRunningTransformAnimationOnCompositor() only needs to be checked
once:

if ((...  && style->isRunningTransformAnimationOnCompositor())
|| (property == CSSPropertyOpacity &&
style->isRunningOpacityAnimationOnCompositor())
|| (property == CSSPropertyWebkitFilter &&
    style->isRunningFilterAnimationOnCompositor())) {

[soonm, 2015/06/30 05:01:08]

Done.

+            || (property == CSSPropertyScale && style->isRunningTransformAnimationOnCompositor())
             || (property == CSSPropertyTransform && style->isRunningTransformAnimationOnCompositor())
+            || (property == CSSPropertyTranslate && style->isRunningTransformAnimationOnCompositor())
             || (property == CSSPropertyWebkitFilter && style->isRunningFilterAnimationOnCompositor())) {
             updateAnimationTiming(element.document(), TimingUpdateOnDemand);
         }