(blink) Propagate scrolling/marginwidth/marginheight property values to child frame.

Source/core/frame/FrameView.cpp

Index: Source/core/frame/FrameView.cpp
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
index cdfa3a72851e8005b125f67f28d907238286c97d..3d9f37ad4101283215df5e2795837a51fc0a7025 100644
--- a/Source/core/frame/FrameView.cpp
+++ b/Source/core/frame/FrameView.cpp
@@ -587,9 +587,13 @@ void FrameView::applyOverflowToViewport(LayoutObject* o, ScrollbarMode& hMode, S
 
 void FrameView::calculateScrollbarModesForLayout(ScrollbarMode& hMode, ScrollbarMode& vMode, ScrollbarModesCalculationStrategy strategy)
 {
-    // FIXME: How do we handle this for OOPI?
-    const HTMLFrameOwnerElement* owner = m_frame->deprecatedLocalOwner();
-    if (owner && (owner->scrollingMode() == ScrollbarAlwaysOff)) {
+    const HTMLFrameOwnerElement* ownerElement = m_frame->deprecatedLocalOwner();

[alexmos, 2015/09/02 21:37:05]

Wouldn't just doing this via m_frame->owner() work for both local and remote
cases?  That would allow us to get rid of the deprecatedLocalOwner call and the
duplicate block.

[lazyboy, 2015/09/15 01:40:32]

Yes, Done.

+    if (ownerElement && (ownerElement->scrollingMode() == ScrollbarAlwaysOff)) {
+        hMode = ScrollbarAlwaysOff;
+        vMode = ScrollbarAlwaysOff;
+        return;
+    }
+    if (!ownerElement && m_frame->owner() && (m_frame->owner()->scrollingMode() == ScrollbarAlwaysOff)) {

[alexmos, 2015/09/02 21:37:05]

Does LocalFrame::createView need a similar fix? 
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[lazyboy, 2015/09/15 01:40:32]

I've added the fix there too similarly.

         hMode = ScrollbarAlwaysOff;
         vMode = ScrollbarAlwaysOff;
         return;