Make <webview> use out-of-process iframe architecture.

content/browser/frame_host/render_frame_host_manager.cc

Index: content/browser/frame_host/render_frame_host_manager.cc
diff --git a/content/browser/frame_host/render_frame_host_manager.cc b/content/browser/frame_host/render_frame_host_manager.cc
index 952b0b0fd17d83e732369f61780de207194629c2..742c0b9b68ced0559f3ef9e4f82865f7d366c4a8 100644
--- a/content/browser/frame_host/render_frame_host_manager.cc
+++ b/content/browser/frame_host/render_frame_host_manager.cc
@@ -144,6 +144,19 @@ RenderWidgetHostView* RenderFrameHostManager::GetRenderWidgetHostView() const {
   return nullptr;
 }
 
+RenderWidgetHostImpl*
+RenderFrameHostManager::GetOuterRenderWidgetHostForKeyboardInput() {
+  if (!ForInnerDelegate())
+    return nullptr;
+
+  FrameTreeNode* outer_contents_frame_tree_node =
+      FrameTreeNode::GloballyFindByID(
+          delegate_->GetOuterDelegateFrameTreeNodeID());
+  return outer_contents_frame_tree_node->parent()
+      ->current_frame_host()
+      ->render_view_host();
+}
+
 RenderFrameProxyHost* RenderFrameHostManager::GetProxyToParent() {
   if (frame_tree_node_->IsMainFrame())
     return NULL;
@@ -160,6 +173,38 @@ RenderFrameProxyHost* RenderFrameHostManager::GetProxyToParent() {
   return iter->second;
 }
 
+RenderFrameProxyHost* RenderFrameHostManager::GetProxyToOuterDelegate() {
+  int outer_contents_frame_tree_node_id =
+      delegate_->GetOuterDelegateFrameTreeNodeID();
+  FrameTreeNode* outer_contents_frame_tree_node =
+      FrameTreeNode::GloballyFindByID(outer_contents_frame_tree_node_id);
+  if (!outer_contents_frame_tree_node ||
+      !outer_contents_frame_tree_node->parent()) {
+    return nullptr;
+  }
+
+  return GetRenderFrameProxyHost(outer_contents_frame_tree_node->parent()
+                                     ->current_frame_host()
+                                     ->GetSiteInstance());
+}
+
+bool RenderFrameHostManager::ForInnerDelegate() {
+  // TODO(lazyboy): We should remove IsMainFrame() check below since any frames

[nasko, 2015/06/11 00:21:28]

Should we? Wouldn't this cause us to skip the ref-counting and proxy creation
code for subframes inside the inner WebContents? I think the only special cased
proxy needs to be the one to the outer WebContents, right?

[lazyboy, 2015/06/11 22:20:29]

Hmmm, I think so.
Out of process subframes inside inner WebContents is something we need
to figure out in detail though, I don't expect them to work as is.
I've changed the TODO.

+  // in the inner WebContents should be treated as a part of inner
+  // WebContents/Delegate.
+  return frame_tree_node_->IsMainFrame() &&
+         delegate_->GetOuterDelegateFrameTreeNodeID() != -1;
+}
+
+void RenderFrameHostManager::RemoveOuterDelegateFrame() {
+  FrameTreeNode* outer_delegate_frame_tree_node =
+      FrameTreeNode::GloballyFindByID(
+          delegate_->GetOuterDelegateFrameTreeNodeID());
+  DCHECK(outer_delegate_frame_tree_node->parent());
+  outer_delegate_frame_tree_node->frame_tree()->RemoveFrame(

[nasko, 2015/06/11 00:21:28]

Shouldn't this work be driven by the chrome/ layer? When the child WebContents
is destroyed?

[lazyboy, 2015/06/11 22:20:29]

This is an <iframe> element in outer WebContents being destroyed, why should
we do this outside of content/?
Do we expose the notion of FTNs outside of content/ so it can be removed from
chrome/?

+      outer_delegate_frame_tree_node);
+}
+
 void RenderFrameHostManager::SetPendingWebUI(const GURL& url, int bindings) {
   pending_web_ui_ = CreateWebUI(url, bindings);
   pending_and_current_web_ui_.reset();
@@ -1647,6 +1692,28 @@ void RenderFrameHostManager::EnsureRenderViewInitialized(
   proxy->set_render_frame_proxy_created(true);
 }
 
+void RenderFrameHostManager::CreateOuterDelegateProxy(
+    SiteInstance* outer_contents_site_instance,
+    RenderFrameHostImpl* render_frame_host) {
+  CHECK(base::CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kSitePerProcess));
+  RenderFrameProxyHost* proxy = new RenderFrameProxyHost(
+      outer_contents_site_instance, render_frame_host->render_view_host(),

[nasko, 2015/06/11 00:21:28]

I thought the idea was to not associate this proxy with any RenderViewHost.
Can't we pass nullptr here?

[lazyboy, 2015/06/11 22:20:29]

That works.
Done.

+      frame_tree_node_);
+  proxy_hosts_[outer_contents_site_instance->GetId()] = proxy;
+
+  render_frame_host->Send(
+      new FrameMsg_SwapOut(render_frame_host->GetRoutingID(),
+                           proxy->GetRoutingID(), false /* is_loading */,
+                           FrameReplicationState()));
+  proxy->set_render_frame_proxy_created(true);
+}
+
+void RenderFrameHostManager::SetRWHViewForInnerContents(
+    RenderWidgetHostView* child_rwhv) {
+  GetProxyToOuterDelegate()->SetChildRWHView(child_rwhv);
+}
+
 bool RenderFrameHostManager::InitRenderView(
     RenderViewHostImpl* render_view_host,
     int opener_route_id,