Propagate scrolling/marginwidth/marginheight property values to child frame.

content/browser/frame_host/frame_tree_node.h

Index: content/browser/frame_host/frame_tree_node.h
diff --git a/content/browser/frame_host/frame_tree_node.h b/content/browser/frame_host/frame_tree_node.h
index 2d033a65db6f56f0f7ad88b946186f2f8457c79c..3ab60358a5d069782b07ec96fefa784ca82cf5ff 100644
--- a/content/browser/frame_host/frame_tree_node.h
+++ b/content/browser/frame_host/frame_tree_node.h
@@ -15,6 +15,7 @@
 #include "content/browser/frame_host/render_frame_host_manager.h"
 #include "content/common/content_export.h"
 #include "content/common/frame_replication_state.h"
+#include "third_party/WebKit/public/web/WebFrameOwnerProperties.h"
 #include "url/gurl.h"
 #include "url/origin.h"
 
@@ -53,7 +54,8 @@ class CONTENT_EXPORT FrameTreeNode {
                 RenderFrameHostManager::Delegate* manager_delegate,
                 blink::WebTreeScopeType scope,
                 const std::string& name,
-                blink::WebSandboxFlags sandbox_flags);
+                blink::WebSandboxFlags sandbox_flags,
+                const blink::WebFrameOwnerProperties& frame_owner_properties);
 
   ~FrameTreeNode();
 
@@ -147,6 +149,15 @@ class CONTENT_EXPORT FrameTreeNode {
     return replication_state_;
   }
 
+  const blink::WebFrameOwnerProperties& frame_owner_properties() {
+    return frame_owner_properties_;
+  }
+
+  void set_frame_owner_properties(
+      const blink::WebFrameOwnerProperties& frame_owner_properties) {
+    frame_owner_properties_ = frame_owner_properties;
+  }
+
   RenderFrameHostImpl* current_frame_host() const {
     return render_manager_.current_frame_host();
   }
@@ -276,6 +287,15 @@ class CONTENT_EXPORT FrameTreeNode {
   // flags when a navigation for this frame commits.
   blink::WebSandboxFlags effective_sandbox_flags_;
 
+  // Tracks the scrolling and margin properties for this frame.  These
+  // properties affect the child renderer but are stored on its parent's
+  // frame element.  When this frame's parent dynamically updates these
+  // properties, we store them here.

[alexmos, 2015/10/02 21:24:19]

nit: last sentence makes it sound like the properties are only stored on dynamic
updates, but they're also stored when the child frame is first created.

[lazyboy, 2015/10/05 22:16:08]

Changed "store" to "update".

+  //
+  // Note that the update only takes effect on the next frame navigation,

[alexmos, 2015/10/02 21:24:19]

nit: s/the update/dynamic updates/, s/takes/take/.  I'd also remove "to be
consistent...", since the code here doesn't do anything to be consistent with
Blink -- that happens on the Blink side in WebFrame::setFrameOwnerProperties.

[lazyboy, 2015/10/05 22:16:08]

Done.

+  // to be consistent with current Blink implementation.
+  blink::WebFrameOwnerProperties frame_owner_properties_;
+
   // Used to track this node's loading progress (from 0 to 1).
   double loading_progress_;