Avoid duplicate math when computing VR scene hierarchy.

chrome/browser/android/vr_shell/ui_elements.h

Index: chrome/browser/android/vr_shell/ui_elements.h
diff --git a/chrome/browser/android/vr_shell/ui_elements.h b/chrome/browser/android/vr_shell/ui_elements.h
index 69ed336d6ba8d3562ab6fe9b4f4b29752a659adb..fc590f993bf5b8934124cacda5e507e69d7eb5a7 100644
--- a/chrome/browser/android/vr_shell/ui_elements.h
+++ b/chrome/browser/android/vr_shell/ui_elements.h
@@ -59,7 +59,7 @@ struct Transform {
 class WorldRectangle {
  public:
   const gvr::Mat4f& TransformMatrix() const;
-  void SetTransform(const Transform& transform);
+  Transform* GetTransform() { return &transform_; }

[mthiesse, 2017/03/27 15:31:16]

nit: maybe use MutableTransform()? Correct style is probably actually
mutable_transform(), using lowercase is encouraged for simple getters/setters.

[cjgrant, 2017/03/27 15:58:24]

Done.  Good call on that!

 
   gvr::Vec3f GetCenter() const;
   gvr::Vec3f GetNormal() const;
@@ -154,6 +154,12 @@ struct ContentRectangle : public WorldRectangle {
 
   int draw_phase = 1;
 
+  // This transform can be used by children to derive position of its parent.
+  Transform inheritable_transform;
+
+  // A flag usable during transformation calculates to avoid duplicate work.
+  bool dirty;
+
  private:
   DISALLOW_COPY_AND_ASSIGN(ContentRectangle);
 };