Convert HTMLFrameOwnerElement and FocusController to use Frame.

Source/core/page/PageSerializer.cpp

Index: Source/core/page/PageSerializer.cpp
diff --git a/Source/core/page/PageSerializer.cpp b/Source/core/page/PageSerializer.cpp
index 6cff82f3532b51622b593c30c1776647c8eedd4c..419df01750cde6d85e68cfc9ef171fe5fe566d36 100644
--- a/Source/core/page/PageSerializer.cpp
+++ b/Source/core/page/PageSerializer.cpp
@@ -154,16 +154,17 @@ void SerializerMarkupAccumulator::appendCustomAttributes(StringBuilder& out, con
         return;
 
     const HTMLFrameOwnerElement& frameOwner = toHTMLFrameOwnerElement(element);
-    LocalFrame* frame = frameOwner.contentFrame();
-    if (!frame)
+    Frame* frame = frameOwner.contentFrame();
+    // RemoteFrames not currently supported here.

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

But eventually, right? Maybe tag with a FIXME?

Another question is how this interacts with WebPageSerializer. It looks like
that doesn't even add a frame to the list of frames to visit (and presumably
serialize). Is this code used by a different codepath in Source/core as well?
I'm assuming so (it looks like it might be called by StyledMarkupAccumulator?)
but it'd be good to have confirmation.

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

I've added a FIXME, but I don't know this area of the code well enough to
comment on your question.

+    if (!frame || !frame->isLocalFrame())
         return;
 
-    KURL url = frame->document()->url();
+    KURL url = toLocalFrame(frame)->document()->url();
     if (url.isValid() && !url.isBlankURL())
         return;
 
     // We need to give a fake location to blank frames so they can be referenced by the serialized frame.
-    url = m_serializer->urlForBlankFrame(frame);
+    url = m_serializer->urlForBlankFrame(toLocalFrame(frame));
     appendAttribute(out, element, Attribute(frameOwnerURLAttributeName(frameOwner), AtomicString(url.string())), namespaces);
 }