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

content/browser/renderer_host/render_view_host_impl.cc

Index: content/browser/renderer_host/render_view_host_impl.cc
diff --git a/content/browser/renderer_host/render_view_host_impl.cc b/content/browser/renderer_host/render_view_host_impl.cc
index a644758232d3cd3fbdc0e505abda4f07b94426a5..da8da7e4b3c5e5582a9c05feff6549385809f43a 100644
--- a/content/browser/renderer_host/render_view_host_impl.cc
+++ b/content/browser/renderer_host/render_view_host_impl.cc
@@ -25,6 +25,7 @@
 #include "base/values.h"
 #include "cc/base/switches.h"
 #include "content/browser/bad_message.h"
+#include "content/browser/browser_plugin/browser_plugin_guest.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/dom_storage/session_storage_namespace_impl.h"
 #include "content/browser/frame_host/frame_tree.h"
@@ -304,6 +305,17 @@ bool RenderViewHostImpl::CreateRenderView(
   if (max_page_id > -1)
     next_page_id = max_page_id + 1;
 
+  int proxy_route_id_to_send = proxy_route_id;
+
+  // Do not instruct the view to create a proxy if this is a swapped out

[nasko, 2015/06/15 23:00:13]

I'm trying to kill "swapped out", so let's not add any more usages of it : ). It
doesn't bring much additional meaning to the sentence that wouldn't exist
without it and there isn't actually any check based on the swapped out state of
the RenderViewHost.

[lazyboy, 2015/06/16 17:59:51]

Good point.
This code path is unused as we do not create RenderViewHost anymore
after switching from RFHM::CreateRenderFrameProxy() to
"new RenderFrameProxyHost()"... in RFHM::CreateOuterDelegateProxy().
So all of this can go away now.
Does this sound right?

+  // RenderView for an inner WebContents.
+  // The proxy will be created using RFHM::SwapFrameWithProxy().

[nasko, 2015/06/15 23:00:13]

There is no SwapFrameWithProxy method anymore.

[lazyboy, 2015/06/16 17:59:51]

See above ^^^.

+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kSitePerProcess) &&
+      BrowserPluginGuest::IsGuest(this)) {
+    proxy_route_id_to_send = MSG_ROUTING_NONE;
+  }
+
   ViewMsg_New_Params params;
   params.renderer_preferences =
       delegate_->GetRendererPrefs(GetProcess()->GetBrowserContext());
@@ -321,7 +333,7 @@ bool RenderViewHostImpl::CreateRenderView(
   params.opener_route_id = opener_route_id;
   params.swapped_out = !is_active_;
   params.replicated_frame_state = replicated_frame_state;
-  params.proxy_routing_id = proxy_route_id;
+  params.proxy_routing_id = proxy_route_id_to_send;
   params.hidden = is_hidden();
   params.never_visible = delegate_->IsNeverVisible();
   params.window_was_created_with_opener = window_was_created_with_opener;