Support script modify iframe scrolling, marginWidth and marginHeight attr

third_party/WebKit/Source/web/WebFrame.cpp

Index: third_party/WebKit/Source/web/WebFrame.cpp
diff --git a/third_party/WebKit/Source/web/WebFrame.cpp b/third_party/WebKit/Source/web/WebFrame.cpp
index 79ce1ac6b9979b00595c6cfcf32f373238e62f1c..7a44cef8a72eac5561b95a34a163ee5025b431a9 100644
--- a/third_party/WebKit/Source/web/WebFrame.cpp
+++ b/third_party/WebKit/Source/web/WebFrame.cpp
@@ -5,6 +5,7 @@
 #include "public/web/WebFrame.h"
 
 #include "bindings/core/v8/WindowProxyManager.h"
+#include "core/HTMLNames.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
@@ -25,6 +26,8 @@
 
 namespace blink {
 
+using namespace HTMLNames;
+
 bool WebFrame::swap(WebFrame* frame) {
   using std::swap;
   Frame* oldFrame = toImplBase()->frame();
@@ -144,6 +147,18 @@ void WebFrame::setFrameOwnerProperties(
   owner->setAllowFullscreen(properties.allowFullscreen);
   owner->setCsp(properties.requiredCsp);
   owner->setDelegatedpermissions(properties.delegatedPermissions);
+
+  Frame* frame = toImplBase()->frame();
+  DCHECK(frame);
+
+  if (frame->isLocalFrame()) {

[bokan, 2016/11/18 20:56:18]

This logic is currently duplicated for Remote and Local frames. I think it'd be
better if we added

Document::didChangeFrameOwnerProperties(marginWidth, marginHeight, scrolling)

and called it from HTMLFrameOwnerBase::setScrollingMode|MarginWidth|MarginHeight
and here (before setting the RemoteFrameOwner properties so document can check
for differences and issue a layout if needed, for example).

+    LocalFrame* localFrame = toLocalFrame(frame);
+    localFrame->document()->body()->setIntegralAttribute(

[bokan, 2016/11/18 20:56:18]

We should only do each of these if the specific attribute changed.

+        marginwidthAttr, properties.marginWidth);
+    localFrame->document()->body()->setIntegralAttribute(
+        marginheightAttr, properties.marginHeight);
+    localFrame->view()->setNeedsLayout();
+  }
 }
 
 WebFrame* WebFrame::opener() const {