Mojo: simple iframe support.

mojo/services/html_viewer/html_document_view.cc

Index: mojo/services/html_viewer/html_document_view.cc
diff --git a/mojo/services/html_viewer/html_document_view.cc b/mojo/services/html_viewer/html_document_view.cc
index 7751aa4e07dc74502b2cf07b10d5b40ee3ae7734..a5b252053c604e61d1dd7e93e97230a494d173c8 100644
--- a/mojo/services/html_viewer/html_document_view.cc
+++ b/mojo/services/html_viewer/html_document_view.cc
@@ -81,10 +81,10 @@ bool CanNavigateLocally(blink::WebFrame* frame,
 HTMLDocumentView::HTMLDocumentView(ServiceProvider* service_provider,
                                    ViewManager* view_manager)
     : view_manager_(view_manager),
+      navigator_host_(service_provider),
       web_view_(NULL),
       root_(NULL),
       repaint_pending_(false),
-      navigator_host_(service_provider),
       weak_factory_(this) {
 }
 
@@ -145,6 +145,31 @@ bool HTMLDocumentView::allowsBrokenNullLayerTreeView() const {
   return true;
 }
 
+blink::WebFrame* HTMLDocumentView::createChildFrame(
+    blink::WebLocalFrame* parent,
+    const blink::WebString& frameName) {
+  blink::WebLocalFrame* web_frame = blink::WebLocalFrame::create(this);
+  parent->appendChild(web_frame);
+  return web_frame;
+}
+
+void HTMLDocumentView::didDisownOpener(blink::WebLocalFrame* frame) {

[darin (slow to review), 2014/08/15 03:26:53]

I suppose these empty methods don't really need to be overridden at this point?

+}
+
+void HTMLDocumentView::frameDetached(blink::WebFrame* frame) {
+  if (frame->parent())
+    frame->parent()->removeChild(frame);
+
+  // |frame| is invalid after here.
+  frame->close();
+}
+
+void HTMLDocumentView::frameFocused() {
+}
+
+void HTMLDocumentView::willClose(blink::WebFrame* frame) {
+}
+
 blink::WebCookieJar* HTMLDocumentView::cookieJar(blink::WebLocalFrame* frame) {
   // TODO(darin): Blink does not fallback to the Platform provided WebCookieJar.
   // Either it should, as it once did, or we should find another solution here.