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

Source/core/html/HTMLFrameElementBase.cpp

Index: Source/core/html/HTMLFrameElementBase.cpp
diff --git a/Source/core/html/HTMLFrameElementBase.cpp b/Source/core/html/HTMLFrameElementBase.cpp
index 75865b6889054698df58dfc4a1fdf6b7b5a251e5..7aa50d4db81a06a17509f4341660aa7e0adf9e23 100644
--- a/Source/core/html/HTMLFrameElementBase.cpp
+++ b/Source/core/html/HTMLFrameElementBase.cpp
@@ -45,9 +45,6 @@ using namespace HTMLNames;
 
 HTMLFrameElementBase::HTMLFrameElementBase(const QualifiedName& tagName, Document& document)
     : HTMLFrameOwnerElement(tagName, document)
-    , m_scrolling(ScrollbarAuto)
-    , m_marginWidth(-1)
-    , m_marginHeight(-1)
 {
 }
 
@@ -120,17 +117,17 @@ void HTMLFrameElementBase::parseAttribute(const QualifiedName& name, const Atomi
         // 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?