Make RenderFrameHostManager swap RenderFrameHosts, not RenderViewHosts.

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index 658e834230dd6fe39ee05f4891563fa7cc618f99..15ea353d7b27dd81d171a6196575560a746b5986 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -48,6 +48,7 @@ RenderFrameHostImpl::RenderFrameHostImpl(
       frame_tree_node_(frame_tree_node),
       routing_id_(routing_id),
       is_swapped_out_(is_swapped_out) {
+  frame_tree_->RegisterRenderFrameHost(this);
   GetProcess()->AddRoute(routing_id_, this);
   g_routing_id_frame_map.Get().insert(std::make_pair(
       RenderFrameHostID(GetProcess()->GetID(), routing_id_),
@@ -60,6 +61,10 @@ RenderFrameHostImpl::~RenderFrameHostImpl() {
       RenderFrameHostID(GetProcess()->GetID(), routing_id_));
   if (delegate_)
     delegate_->RenderFrameDeleted(this);
+
+  // Decrement the refcount of the frame's RenderViewHost, allowing it to shut

[nasko, 2013/12/12 02:22:12]

nit: The comment as worded reveals inner workings of register/unregister. The
fact that we keep ref counts is technically encapsulated in the frame tree. I
don't have a good suggestion though, just didn't seem right to talk about ref
counts here.

[Charlie Reis, 2013/12/12 18:19:30]

Done.

+  // down if it is no longer needed.
+  frame_tree_->UnregisterRenderFrameHost(this);
 }
 
 int RenderFrameHostImpl::GetRoutingID() {