Do not push properties that are animated on impl only.

cc/layer_unittest.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/layer.h"
 
 #include "cc/keyframed_animation_curve.h"
 #include "cc/layer_impl.h"
 #include "cc/layer_painter.h"
 #include "cc/layer_tree_host.h"
 #include "cc/math_util.h"
 #include "cc/single_thread_proxy.h"
+#include "cc/test/animation_test_common.h"
 #include "cc/test/fake_impl_proxy.h"
 #include "cc/test/fake_layer_tree_host_client.h"
 #include "cc/test/fake_layer_tree_host_impl.h"
 #include "cc/test/geometry_test_utils.h"
 #include "cc/thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/transform.h"
 
 using ::testing::AnyNumber;
@@ skipping 613 matching lines @@
 
     testLayer->setOpacity(0.5);
 
     EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
 
     testLayer->pushPropertiesTo(implLayer.get());
 
     EXPECT_TRUE(implLayer->layerSurfacePropertyChanged());
 }
 
+TEST_F(LayerTest, verifyPushPropertiesDoesNotCauseSurfacePropertyChangedDuringImplOnlyTransformAnimation)
+{
+    scoped_refptr<Layer> testLayer = Layer::create();
+    scoped_ptr<LayerImpl> implLayer = LayerImpl::create(m_hostImpl.activeTree(), 1);
+
+    scoped_ptr<AnimationRegistrar> registrar = AnimationRegistrar::create();
+    implLayer->layerAnimationController()->setAnimationRegistrar(registrar.get());
+
+    addAnimatedTransformToController(*implLayer->layerAnimationController(), 1.0, 0, 100);
+
+    gfx::Transform transform;
+    transform.Rotate(45.0);
+    testLayer->setTransform(transform);
+
+    EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
+    testLayer->pushPropertiesTo(implLayer.get());
+    EXPECT_TRUE(implLayer->layerSurfacePropertyChanged());
+
+    implLayer->resetAllChangeTrackingForSubtree();
+    addAnimatedTransformToController(*implLayer->layerAnimationController(), 1.0, 0, 100);
+    implLayer->layerAnimationController()->getAnimation(Animation::Transform)->setIsImplOnly(true);
+    transform.Rotate(45.0);
+    testLayer->setTransform(transform);
+
+    EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
+    testLayer->pushPropertiesTo(implLayer.get());
+    EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
+}
+
+TEST_F(LayerTest, verifyPushPropertiesDoesNotCauseSurfacePropertyChangedDuringImplOnlyOpacityAnimation)
+{
+    scoped_refptr<Layer> testLayer = Layer::create();
+    scoped_ptr<LayerImpl> implLayer = LayerImpl::create(m_hostImpl.activeTree(), 1);
+
+    scoped_ptr<AnimationRegistrar> registrar = AnimationRegistrar::create();
+    implLayer->layerAnimationController()->setAnimationRegistrar(registrar.get());
+
+    addOpacityTransitionToController(*implLayer->layerAnimationController(), 1.0, 0.3f, 0.7f, false);
+
+    testLayer->setOpacity(0.5f);
+
+    EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
+    testLayer->pushPropertiesTo(implLayer.get());
+    EXPECT_TRUE(implLayer->layerSurfacePropertyChanged());
+
+    implLayer->resetAllChangeTrackingForSubtree();
+    addOpacityTransitionToController(*implLayer->layerAnimationController(), 1.0, 0.3f, 0.7f, false);
+    implLayer->layerAnimationController()->getAnimation(Animation::Opacity)->setIsImplOnly(true);
+    testLayer->setOpacity(0.75f);
+
+    EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
+    testLayer->pushPropertiesTo(implLayer.get());
+    EXPECT_FALSE(implLayer->layerSurfacePropertyChanged());
+}
+
 TEST_F(LayerTest, maskAndReplicaHasParent)
 {
     scoped_refptr<Layer> parent = Layer::create();
     scoped_refptr<Layer> child = Layer::create();
     scoped_refptr<Layer> mask = Layer::create();
     scoped_refptr<Layer> replica = Layer::create();
     scoped_refptr<Layer> replicaMask = Layer::create();
     scoped_refptr<Layer> maskReplacement = Layer::create();
     scoped_refptr<Layer> replicaReplacement = Layer::create();
     scoped_refptr<Layer> replicaMaskReplacement = Layer::create();
@@ skipping 255 matching lines @@
     layer->setLayerTreeHost(layerTreeHost.get());
     assertLayerTreeHostMatchesForSubtree(layer.get(), layerTreeHost.get());
 
     // Case 3: with a LayerTreeHost where accelerated animation is disabled, the
     // animation should be rejected.
     EXPECT_FALSE(addTestAnimation(layer.get()));
 }
 
 }  // namespace
 }  // namespace cc