diff options
Diffstat (limited to 'WebCore/html/HTMLFrameElementBase.cpp')
-rw-r--r-- | WebCore/html/HTMLFrameElementBase.cpp | 93 |
1 files changed, 7 insertions, 86 deletions
diff --git a/WebCore/html/HTMLFrameElementBase.cpp b/WebCore/html/HTMLFrameElementBase.cpp index a4f0a5e..1bc4995 100644 --- a/WebCore/html/HTMLFrameElementBase.cpp +++ b/WebCore/html/HTMLFrameElementBase.cpp @@ -50,18 +50,17 @@ HTMLFrameElementBase::HTMLFrameElementBase(const QualifiedName& tagName, Documen , m_scrolling(ScrollbarAuto) , m_marginWidth(-1) , m_marginHeight(-1) - , m_noResize(false) , m_viewSource(false) , m_shouldOpenURLAfterAttach(false) { } -bool HTMLFrameElementBase::isURLAllowed(const AtomicString& URLString) const +bool HTMLFrameElementBase::isURLAllowed() const { - if (URLString.isEmpty()) + if (m_URL.isEmpty()) return true; - const KURL& completeURL = document()->completeURL(URLString); + const KURL& completeURL = document()->completeURL(m_URL); // Don't allow more than 200 total frames in a set. This seems // like a reasonable upper bound, and otherwise mutually recursive @@ -93,7 +92,7 @@ void HTMLFrameElementBase::openURL() { ASSERT(!m_frameName.isEmpty()); - if (!isURLAllowed(m_URL)) + if (!isURLAllowed()) return; if (m_URL.isEmpty()) @@ -127,9 +126,6 @@ void HTMLFrameElementBase::parseMappedAttribute(MappedAttribute *attr) } else if (attr->name() == marginheightAttr) { m_marginHeight = attr->value().toInt(); // FIXME: If we are already attached, this has no effect. - } else if (attr->name() == noresizeAttr) { - m_noResize = true; - // FIXME: If we are already attached, this has no effect. } else if (attr->name() == scrollingAttr) { // Auto and yes both simply mean "allow scrolling." No means "don't allow scrolling." if (equalIgnoringCase(attr->value(), "auto") || equalIgnoringCase(attr->value(), "yes")) @@ -201,7 +197,7 @@ void HTMLFrameElementBase::attach() KURL HTMLFrameElementBase::location() const { - return src(); + return document()->completeURL(getAttribute(srcAttr)); } void HTMLFrameElementBase::setLocation(const String& str) @@ -216,9 +212,9 @@ void HTMLFrameElementBase::setLocation(const String& str) openURL(); } -bool HTMLFrameElementBase::isFocusable() const +bool HTMLFrameElementBase::supportsFocus() const { - return renderer(); + return true; } void HTMLFrameElementBase::setFocus(bool received) @@ -233,81 +229,6 @@ bool HTMLFrameElementBase::isURLAttribute(Attribute *attr) const return attr->name() == srcAttr; } -String HTMLFrameElementBase::frameBorder() const -{ - return getAttribute(frameborderAttr); -} - -void HTMLFrameElementBase::setFrameBorder(const String &value) -{ - setAttribute(frameborderAttr, value); -} - -String HTMLFrameElementBase::longDesc() const -{ - return getAttribute(longdescAttr); -} - -void HTMLFrameElementBase::setLongDesc(const String &value) -{ - setAttribute(longdescAttr, value); -} - -String HTMLFrameElementBase::marginHeight() const -{ - return getAttribute(marginheightAttr); -} - -void HTMLFrameElementBase::setMarginHeight(const String &value) -{ - setAttribute(marginheightAttr, value); -} - -String HTMLFrameElementBase::marginWidth() const -{ - return getAttribute(marginwidthAttr); -} - -void HTMLFrameElementBase::setMarginWidth(const String &value) -{ - setAttribute(marginwidthAttr, value); -} - -String HTMLFrameElementBase::name() const -{ - return getAttribute(nameAttr); -} - -void HTMLFrameElementBase::setName(const String &value) -{ - setAttribute(nameAttr, value); -} - -void HTMLFrameElementBase::setNoResize(bool noResize) -{ - setAttribute(noresizeAttr, noResize ? "" : 0); -} - -String HTMLFrameElementBase::scrolling() const -{ - return getAttribute(scrollingAttr); -} - -void HTMLFrameElementBase::setScrolling(const String &value) -{ - setAttribute(scrollingAttr, value); -} - -KURL HTMLFrameElementBase::src() const -{ - return document()->completeURL(getAttribute(srcAttr)); -} - -void HTMLFrameElementBase::setSrc(const String &value) -{ - setAttribute(srcAttr, value); -} - int HTMLFrameElementBase::width() const { if (!renderer()) |