Create RenderFrameProxyHost at time of swap-out instead of commit.

content/browser/frame_host/render_frame_proxy_host.cc

Index: content/browser/frame_host/render_frame_proxy_host.cc
diff --git a/content/browser/frame_host/render_frame_proxy_host.cc b/content/browser/frame_host/render_frame_proxy_host.cc
index e965fdac528e19cfa0e81319ca87271805d36fd8..f0ed1d83e1741e94f00ee4ecf54eba4749ded1fd 100644
--- a/content/browser/frame_host/render_frame_proxy_host.cc
+++ b/content/browser/frame_host/render_frame_proxy_host.cc
@@ -4,16 +4,45 @@
 
 #include "content/browser/frame_host/render_frame_proxy_host.h"
 
+#include "content/browser/frame_host/frame_tree_node.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/site_instance_impl.h"
+#include "content/common/frame_messages.h"
+#include "ipc/ipc_message.h"
 
 namespace content {
 
-RenderFrameProxyHost::RenderFrameProxyHost(
-    scoped_ptr<RenderFrameHostImpl> render_frame_host)
-    : site_instance_(render_frame_host->GetSiteInstance()),
-      render_frame_host_(render_frame_host.Pass()) {}
+RenderFrameProxyHost::RenderFrameProxyHost(SiteInstance* site_instance,
+                                           FrameTreeNode* frame_tree_node)
+    : routing_id_(site_instance->GetProcess()->GetNextRoutingID()),
+      site_instance_(site_instance),
+      frame_tree_node_(frame_tree_node) {
+  GetProcess()->AddRoute(routing_id_, this);
+}
 
-RenderFrameProxyHost::~RenderFrameProxyHost() {}
+RenderFrameProxyHost::~RenderFrameProxyHost() {
+  GetProcess()->RemoveRoute(routing_id_);
+}
+
+void RenderFrameProxyHost::DeleteRendererProxy() {
+  Send(new FrameMsg_DeleteProxy(routing_id_));
+}
+
+bool RenderFrameProxyHost::Send(IPC::Message *msg) {
+  // TODO(nasko): For now, RenderFrameHost uses this object to send IPC messages
+  // while in swapped out state. Once the swapped out state is removed, it
+  // no longer needs to overwrite the routing id.
+  msg->set_routing_id(routing_id_);
+  return GetProcess()->Send(msg);
+}
+
+bool RenderFrameProxyHost::OnMessageReceived(const IPC::Message& msg) {
+  // TODO(nasko): Remove this forward as soon as swapping out of
+  // RenderFrameHosts is removed.
+  if (render_frame_host_.get()) {

[Charlie Reis, 2014/05/15 00:32:50]

nit: No braces needed.

[nasko, 2014/05/15 18:47:13]

Done.

+    return render_frame_host_->OnMessageReceived(msg);
+  }
+  return false;
+}
 
 }  // namespace content