Convert HTMLFrameOwnerElement and FocusController to use Frame.

Source/core/page/PageSerializer.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 136 matching lines @@
 
     // FIXME: For object (plugins) tags and video tag we could replace them by an image of their current contents.
 }
 
 void SerializerMarkupAccumulator::appendCustomAttributes(StringBuilder& out, const Element& element, Namespaces* namespaces)
 {
     if (!element.isFrameOwnerElement())
         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);
 }
 
 void SerializerMarkupAccumulator::appendEndTag(const Node& node)
 {
     if (node.isElementNode() && !shouldIgnoreElement(toElement(node)))
         MarkupAccumulator::appendEndTag(node);
 }
 
 PageSerializer::PageSerializer(Vector<SerializedResource>* resources)
@@ skipping 203 matching lines @@
     if (iter != m_blankFrameURLs.end())
         return iter->value;
     String url = "wyciwyg://frame/" + String::number(m_blankFrameCounter++);
     KURL fakeURL(ParsedURLString, url);
     m_blankFrameURLs.add(frame, fakeURL);
 
     return fakeURL;
 }
 
 }