TransformTree::SetRootTransformsAndScales should set 'needs_update' on transform tree only if we ar…

cc/trees/property_tree_unittest.cc

Index: cc/trees/property_tree_unittest.cc
diff --git a/cc/trees/property_tree_unittest.cc b/cc/trees/property_tree_unittest.cc
index d85f0be7affa5460961eb92216d35ea9795664d5..b2f5a78d41464fec090eea1226cdf8b141f58716 100644
--- a/cc/trees/property_tree_unittest.cc
+++ b/cc/trees/property_tree_unittest.cc
@@ -421,6 +421,28 @@ class PropertyTreeTestComputeTransformRoot : public PropertyTreeTest {
 DIRECT_AND_SERIALIZED_PROPERTY_TREE_TEST_F(
     PropertyTreeTestComputeTransformRoot);
 
+class PropertyTreeTestSetNeedsUpdate : public PropertyTreeTest {
+ protected:
+  void StartTest() override {
+    PropertyTrees property_trees;
+    TransformTree& tree = property_trees.transform_tree;
+    TransformNode contents_root;
+    contents_root.source_node_id = 0;
+    contents_root.id = tree.Insert(contents_root, 0);
+    tree.SetTargetId(contents_root.id, 0);
+    SetupTransformTreeForTest(&tree);
+
+    EXPECT_FALSE(tree.needs_update());
+    tree.SetRootTransformsAndScales(0.6f, 1.f, gfx::Transform(), gfx::PointF());
+    EXPECT_TRUE(tree.needs_update());
+    tree.set_needs_update(false);
+    tree.SetRootTransformsAndScales(0.6f, 1.f, gfx::Transform(), gfx::PointF());
+    EXPECT_FALSE(tree.needs_update());
+  }
+};
+
+DIRECT_AND_SERIALIZED_PROPERTY_TREE_TEST_F(PropertyTreeTestSetNeedsUpdate);
+
 class PropertyTreeTestComputeTransformChild : public PropertyTreeTest {
  protected:
   void StartTest() override {