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

content/browser/frame_host/render_frame_proxy_host.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #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