cc : Split TransformTree::ComputeTransform

cc/trees/property_tree.h

Index: cc/trees/property_tree.h
diff --git a/cc/trees/property_tree.h b/cc/trees/property_tree.h
index 396c4cf30f68eebc5b18fdef8651495a3b17cae2..71df6b1fe3841fee161706fd6056f1bbf55fa36a 100644
--- a/cc/trees/property_tree.h
+++ b/cc/trees/property_tree.h
@@ -154,6 +154,10 @@ class CC_EXPORT TransformTree final : public PropertyTree<TransformNode> {
                         int dest_id,
                         gfx::Transform* transform) const;
 
+  bool ComputeTranslation(int source_id,

[ajuma, 2016/07/27 19:00:31]

Please add a comment explaining what this does and the conditions that source_id
and dest_id must satisfy (e.g. that the transform between them must be at most a
translation).

+                          int dest_id,
+                          gfx::Transform* transform) const;
+
   void ResetChangeTracking();
   // Updates the parent, target, and screen space transforms and snapping.
   void UpdateTransforms(int id);
@@ -245,10 +249,9 @@ class CC_EXPORT TransformTree final : public PropertyTree<TransformNode> {
   void FromProtobuf(const proto::PropertyTree& proto,
                     std::unordered_map<int, int>* node_id_to_index_map);
 
-  // Computes the combined transform between |source_id| and |dest_id| and
-  // returns false if the inverse of a singular transform was used. These two
-  // nodes must be on the same ancestor chain.
-  bool CombineTransformsBetween(int source_id,
+  // Computes the combined transform between |source_id| and |dest_id|. These
+  // two nodes must be on the same ancestor chain.
+  void CombineTransformsBetween(int source_id,
                                 int dest_id,
                                 gfx::Transform* transform) const;
 
@@ -591,6 +594,10 @@ class CC_EXPORT PropertyTrees final {
                                 int effect_id,
                                 gfx::Transform* transform) const;
 
+  bool ComputeTransformFromTarget(int transform_id,
+                                  int effect_id,
+                                  gfx::Transform* transform) const;
+
  private:
   gfx::Vector2dF inner_viewport_container_bounds_delta_;
   gfx::Vector2dF outer_viewport_container_bounds_delta_;