Convert HTMLFrameOwnerElement and FocusController to use Frame.

Source/core/html/HTMLFrameOwnerElement.cpp

Index: Source/core/html/HTMLFrameOwnerElement.cpp
diff --git a/Source/core/html/HTMLFrameOwnerElement.cpp b/Source/core/html/HTMLFrameOwnerElement.cpp
index 5ebb4b8f4c7bf91a31cee1f4b671af40f206152d..a28071d57451c4df16524876f76660cce4f9cd54 100644
--- a/Source/core/html/HTMLFrameOwnerElement.cpp
+++ b/Source/core/html/HTMLFrameOwnerElement.cpp
@@ -51,7 +51,7 @@ RenderPart* HTMLFrameOwnerElement::renderPart() const
     return toRenderPart(renderer());
 }
 
-void HTMLFrameOwnerElement::setContentFrame(LocalFrame& frame)
+void HTMLFrameOwnerElement::setContentFrame(Frame& frame)
 {
     // Make sure we will not end up with two frames referencing the same owner element.
     ASSERT(!m_contentFrame || m_contentFrame->ownerElement() != this);
@@ -80,9 +80,10 @@ void HTMLFrameOwnerElement::disconnectContentFrame()
     // unload event in the subframe which could execute script that could then
     // reach up into this document and then attempt to look back down. We should
     // see if this behavior is really needed as Gecko does not allow this.
-    if (LocalFrame* frame = contentFrame()) {
-        RefPtr<LocalFrame> protect(frame);
-        frame->loader().frameDetached();
+    if (Frame* frame = contentFrame()) {
+        RefPtr<Frame> protect(frame);
+        if (frame->isLocalFrame())
+            toLocalFrame(frame)->loader().frameDetached();

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

Do we need a FIXME here? How are we planning on triggering the detach of remote
frames?

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

I don't know exactly what attachment and detachment will look like on
RemoteFrames; figuring out the details is one of our next tasks. I don't think
we'll need a FIXME though because we can plumb the right calls when we no what
methods RemoteFrame will have for the purpose.

         frame->disconnectOwnerElement();
     }
 }
@@ -124,8 +125,8 @@ SVGDocument* HTMLFrameOwnerElement::getSVGDocument(ExceptionState& exceptionStat
 bool HTMLFrameOwnerElement::loadOrRedirectSubframe(const KURL& url, const AtomicString& frameName, bool lockBackForwardList)
 {
     RefPtr<LocalFrame> parentFrame = document().frame();
-    if (contentFrame()) {
-        contentFrame()->navigationScheduler().scheduleLocationChange(&document(), url.string(), Referrer(document().outgoingReferrer(), document().referrerPolicy()), lockBackForwardList);
+    if (contentFrame() && contentFrame()->isLocalFrame()) {

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

Similar comment to my earlier comment in HTMLFrameElementBase::openURL.

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

I've added a FIXME here. I don't know yet if RemoteFrames will have
NavigationSchedulers, or if they can just do straight navigations.

+        toLocalFrame(contentFrame())->navigationScheduler().scheduleLocationChange(&document(), url.string(), Referrer(document().outgoingReferrer(), document().referrerPolicy()), lockBackForwardList);
         return true;
     }