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

Source/core/html/HTMLFrameElementBase.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Simon Hausmann (hausmann@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2006, 2008, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 27 matching lines @@
 #include "core/loader/FrameLoader.h"
 #include "core/page/FocusController.h"
 #include "core/page/Page.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 HTMLFrameElementBase::HTMLFrameElementBase(const QualifiedName& tagName, Document& document)
     : HTMLFrameOwnerElement(tagName, document)
-    , m_scrolling(ScrollbarAuto)
-    , m_marginWidth(-1)
-    , m_marginHeight(-1)
 {
 }
 
 bool HTMLFrameElementBase::isURLAllowed() const
 {
     if (m_URL.isEmpty())
         return true;
 
     const KURL& completeURL = document().completeURL(m_URL);
 
@@ skipping 52 matching lines @@
     } else if (name == idAttr) {
         // Important to call through to base for the id attribute so the hasID bit gets set.
         HTMLFrameOwnerElement::parseAttribute(name, value);
         m_frameName = value;
     } else if (name == nameAttr) {
         m_frameName = value;
         // FIXME: If we are already attached, this doesn't actually change the frame's name.
         // FIXME: If we are already attached, this doesn't check for frame name
         // conflicts and generate a unique frame name.
     } else if (name == marginwidthAttr) {
-        m_marginWidth = value.toInt();
+        setMarginWidth(value.toInt());
         // FIXME: If we are already attached, this has no effect.
     } else if (name == marginheightAttr) {
-        m_marginHeight = value.toInt();
+        setMarginHeight(value.toInt());
         // FIXME: If we are already attached, this has no effect.
     } else if (name == scrollingAttr) {
         // Auto and yes both simply mean "allow scrolling." No means "don't allow scrolling."
         if (equalIgnoringCase(value, "auto") || equalIgnoringCase(value, "yes"))
-            m_scrolling = ScrollbarAuto;
+            setScrollingMode(ScrollbarAuto);
         else if (equalIgnoringCase(value, "no"))
-            m_scrolling = ScrollbarAlwaysOff;
+            setScrollingMode(ScrollbarAlwaysOff);
         // FIXME: If we are already attached, this has no effect.
     } else if (name == onbeforeunloadAttr) {
         // FIXME: should <frame> elements have beforeunload handlers?
         setAttributeEventListener(EventTypeNames::beforeunload, createAttributeEventListener(this, name, value, eventParameterName()));
     } else {
         HTMLFrameOwnerElement::parseAttribute(name, value);
     }
 }
 
 void HTMLFrameElementBase::setNameAndOpenURL()
@@ skipping 78 matching lines @@
 void HTMLFrameElementBase::defaultEventHandler(Event* event)
 {
     if (contentFrame() && contentFrame()->isRemoteFrame()) {
         toRemoteFrame(contentFrame())->forwardInputEvent(event);
         return;
     }
     HTMLFrameOwnerElement::defaultEventHandler(event);
 }
 
 } // namespace blink