Support script modify iframe scrolling, marginWidth and marginHeight attr

third_party/WebKit/Source/core/dom/Document.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
@@ skipping 4292 matching lines @@
 }
 
 HTMLFrameOwnerElement* Document::localOwner() const {
   if (!frame())
     return 0;
   // FIXME: This probably breaks the attempts to layout after a load is finished
   // in implicitClose(), and probably tons of other things...
   return frame()->deprecatedLocalOwner();
 }
 
+void Document::didChangeFrameOwnerProperties(int marginWidth,
+                                             int marginHeight,
+                                             ScrollbarMode scrollingMode) {
+  DCHECK(localOwner());

[bokan, 2016/11/21 16:11:36]

The owner can be remote. You need some way to get the FrameOwner regardless if
its RemoteFrameOwner or HTMLFrameOwnerElement. OOPIF reviewer might be able to
help with the right way to do this.

[alexmos, 2016/11/21 17:56:56]

Just using frame()->owner() instead should work here.

+
+  if (marginWidth != localOwner()->marginWidth())
+    body()->setIntegralAttribute(marginwidthAttr, marginWidth);
+  if (marginHeight != localOwner()->marginHeight())
+    body()->setIntegralAttribute(marginheightAttr, marginHeight);
+  if (scrollingMode != localOwner()->scrollingMode())
+    view()->setNeedsLayout();
+}
+
 bool Document::isInInvisibleSubframe() const {
   if (!localOwner())
     return false;  // this is a local root element
 
   // TODO(bokan): This looks like it doesn't work in OOPIF.
   DCHECK(frame());
   return frame()->ownerLayoutItem().isNull();
 }
 
 String Document::cookie(ExceptionState& exceptionState) const {
@@ skipping 2117 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->url().getString().utf8().data());
 }
 #endif