Convert HTMLFrameOwnerElement and FocusController to use 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 74 matching lines @@
     // Support for <frame src="javascript:string">
     KURL scriptURL;
     KURL url = document().completeURL(m_URL);
     if (protocolIsJavaScript(m_URL)) {
         scriptURL = url;
         url = blankURL();
     }
 
     if (!loadOrRedirectSubframe(url, m_frameName, lockBackForwardList))
         return;
-    if (!contentFrame() || scriptURL.isEmpty())
+    if (!contentFrame() || scriptURL.isEmpty() || !contentFrame()->isLocalFrame())

[dcheng, 2014/03/17 18:17:44]

Do we need a FIXME here? Isn't location settable on a remote frame
(http://www.whatwg.org/specs/web-apps/current-work/multipage/browsers.html#sec...

[kenrb, 2014/03/18 20:01:19]

This check only prevents JavaScript URLs from running, which I believe is
correct behavior on a cross-origin frame. AFAICT we are okay to just ignore it.

         return;
-    contentFrame()->script().executeScriptIfJavaScriptURL(scriptURL);
+    toLocalFrame(contentFrame())->script().executeScriptIfJavaScriptURL(scriptURL);
 }
 
 void HTMLFrameElementBase::parseAttribute(const QualifiedName& name, const AtomicString& value)
 {
     if (name == srcdocAttr)
         setLocation("about:srcdoc");
     else if (name == srcAttr && !fastHasAttribute(srcdocAttr))
         setLocation(stripLeadingAndTrailingHTMLSpaces(value));
     else if (isIdAttributeName(name)) {
         // Important to call through to base for the id attribute so the hasID bit gets set.
@@ skipping 49 matching lines @@
         return;
 
     setNameAndOpenURL();
 }
 
 void HTMLFrameElementBase::attach(const AttachContext& context)
 {
     HTMLFrameOwnerElement::attach(context);
 
     if (RenderPart* part = renderPart()) {
-        if (LocalFrame* frame = contentFrame())
-            part->setWidget(frame->view());
+        if (Frame* frame = contentFrame()) {
+            if (frame->isLocalFrame())
+                part->setWidget(toLocalFrame(frame)->view());
+        }
     }
 }
 
 KURL HTMLFrameElementBase::location() const
 {
     if (fastHasAttribute(srcdocAttr))
         return KURL(ParsedURLString, "about:srcdoc");
     return document().completeURL(getAttribute(srcAttr));
 }
 
@@ skipping 42 matching lines @@
 
 int HTMLFrameElementBase::height()
 {
     document().updateLayoutIgnorePendingStylesheets();
     if (!renderBox())
         return 0;
     return renderBox()->height();
 }
 
 } // namespace WebCore