Recomposite transform animations when CSS zoom changes

Source/core/animation/css/CSSAnimations.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 223 matching lines @@
         Animation& animation = *runningAnimation->animation;
         if (animation.effect() && animation.effect()->isKeyframeEffect()) {
             EffectModel* model = toKeyframeEffect(animation.effect())->model();
             if (model && model->isKeyframeEffectModel()) {
                 KeyframeEffectModelBase* keyframeEffect = toKeyframeEffectModelBase(model);
                 if (keyframeEffect->hasSyntheticKeyframes() && keyframeEffect->snapshotNeutralCompositorKeyframes(element, oldStyle, style))
                     update.updateCompositorKeyframes(&animation);
             }
         }
     }
+
+    if (oldStyle.effectiveZoom() != style.effectiveZoom()) {

[dstockwell, 2015/09/17 04:07:13]

early out if !oldStyle.isRunningTransformAnimationOnCompositor

[alancutter (OOO until 2018), 2015/09/17 04:21:45]

Done. Added early out of entire function for when no compositor animations are
present.

+        for (auto& entry : elementAnimations->animations()) {
+            Animation& animation = *entry.key;
+            if (animation.effect() && animation.effect()->isKeyframeEffect()) {
+                EffectModel* model = toKeyframeEffect(animation.effect())->model();
+                if (model && model->isKeyframeEffectModel()) {
+                    KeyframeEffectModelBase* keyframeEffect = toKeyframeEffectModelBase(model);
+                    if (keyframeEffect->affects(PropertyHandle(CSSPropertyTransform)) && keyframeEffect->snapshotAllCompositorKeyframes(element, &style))
+                        update.updateCompositorKeyframes(&animation);
+                }
+            }
+        }
+    }
 }
 
 void CSSAnimations::calculateAnimationUpdate(CSSAnimationUpdate& update, const Element* animatingElement, Element& element, const ComputedStyle& style, ComputedStyle* parentStyle, StyleResolver* resolver)
 {
     const ElementAnimations* elementAnimations = animatingElement ? animatingElement->elementAnimations() : nullptr;
 
     bool isAnimationStyleChange = elementAnimations && elementAnimations->isAnimationStyleChange();
 
 #if !ENABLE(ASSERT)
     // If we're in an animation style change, no animations can have started, been cancelled or changed play state.
@@ skipping 542 matching lines @@
 }
 
 DEFINE_TRACE(CSSAnimations)
 {
     visitor->trace(m_transitions);
     visitor->trace(m_pendingUpdate);
     visitor->trace(m_animations);
 }
 
 } // namespace blink