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

third_party/WebKit/public/web/WebLocalFrame.h

 // Copyright 2014 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.
 
 #ifndef WebLocalFrame_h
 #define WebLocalFrame_h
 
 #include "WebFrame.h"
 #include "WebFrameLoadType.h"
 
@@ skipping 34 matching lines @@
 
     // Returns the frame inside a given frame or iframe element. Returns 0 if
     // the given element is not a frame, iframe or if the frame is empty.
     BLINK_EXPORT static WebLocalFrame* fromFrameOwnerElement(const WebElement&);
 
     // Initialization ---------------------------------------------------------
 
     // Used when we might swap from a remote frame to a local frame.
     // Creates a provisional, semi-attached frame that will be fully
     // swapped into the frame tree if it commits.
-    virtual void initializeToReplaceRemoteFrame(WebRemoteFrame*, const WebString& name, WebSandboxFlags) = 0;
+    virtual void initializeToReplaceRemoteFrame(WebRemoteFrame*, const WebString& name, WebSandboxFlags, const WebFrameOwnerProperties&) = 0;
 
     virtual void setAutofillClient(WebAutofillClient*) = 0;
     virtual WebAutofillClient* autofillClient() = 0;
     virtual void setDevToolsAgentClient(WebDevToolsAgentClient*) = 0;
     virtual WebDevToolsAgent* devToolsAgent() = 0;
 
+    // Basic properties ---------------------------------------------------
+
+    // Updates the scrolling and margin properties in the frame's FrameOwner.
+    // This is used when this frame's parent is in another process and it
+    // dynamically updates these properties.
+    // TODO(dcheng): Currently, the update only takes effect on next frame

[dcheng, 2015/10/21 21:04:01]

What's this TODO for? =)

[lazyboy, 2015/10/23 21:19:35]

This corresponds to the FIXMEs in 
https://codereview.chromium.org/1307013004/diff/520001/third_party/WebKit/Sou...

} else if (name == marginwidthAttr) {	 131     } else if (name ==
marginwidthAttr) {
 123         m_marginWidth = value.toInt();	 132        
setMarginWidth(value.toInt());
 124         // FIXME: If we are already attached, this has no effect.

, etc.
Maybe a TODO here is superfluous?

+    // navigation.  This matches the in-process frame behavior.
+    virtual void setFrameOwnerProperties(const WebFrameOwnerProperties&) = 0;
+
     // Navigation Ping --------------------------------------------------------
     virtual void sendPings(const WebNode& contextNode, const WebURL& destinationURL) = 0;
 
     // Navigation ----------------------------------------------------------
 
     // Returns a WebURLRequest corresponding to the load of the WebHistoryItem.
     virtual WebURLRequest requestFromHistoryItem(const WebHistoryItem&, WebURLRequest::CachePolicy)
         const = 0;
 
     // Returns a WebURLRequest corresponding to the reload of the current
@@ skipping 108 matching lines @@
     // Returns the effective sandbox flags which are inherited from their parent frame.
     virtual WebSandboxFlags effectiveSandboxFlags() const = 0;
 
 protected:
     explicit WebLocalFrame(WebTreeScopeType scope) : WebFrame(scope) { }
 };
 
 } // namespace blink
 
 #endif // WebLocalFrame_h