Add ref-count on RenderViewHost for each new RenderFrameProxyHost.

content/browser/frame_host/frame_tree.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_H_
 #define CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_H_
 
 #include <string>
 
 #include "base/callback.h"
@@ skipping 103 matching lines @@
                                            int routing_id,
                                            int main_frame_routing_id,
                                            bool swapped_out,
                                            bool hidden);
 
   // Returns the existing RenderViewHost for a new RenderFrameHost.
   // There should always be such a RenderViewHost, because the main frame
   // RenderFrameHost for each SiteInstance should be created before subframes.
   RenderViewHostImpl* GetRenderViewHost(SiteInstance* site_instance);
 
-  // Keeps track of which RenderFrameHosts are using each RenderViewHost.  When
-  // the number drops to zero, we call Shutdown on the RenderViewHost.
-  void RegisterRenderFrameHost(RenderFrameHostImpl* render_frame_host);
-  void UnregisterRenderFrameHost(RenderFrameHostImpl* render_frame_host);
+  // Keeps track of which RenderFrameHosts and RenderFrameProxyHosts are using
+  // each RenderViewHost.  When the number drops to zero, we call Shutdown on
+  // the RenderViewHost.
+  void RegisterRenderViewHost(RenderViewHostImpl* render_view_host);
+  void UnregisterRenderViewHost(RenderViewHostImpl* render_view_host);

[Charlie Reis, 2015/05/20 23:05:00]

We should pick a better name. "Register" made sense when it referred to the
thing keeping RVH alive, but now the name is about the RVH itself.  Maybe
IncrementRenderViewHostRefCount?  That's a bit awkward, so I'm open to
alternatives.

[nasko, 2015/05/21 16:22:12]

Renamed to keep the same pattern with RefCounted - AddRVHRef/ReleaseRVHRef.

 
   // This is only meant to be called by FrameTreeNode. Triggers calling
   // the listener installed by SetFrameRemoveListener.
   void FrameRemoved(FrameTreeNode* frame);
 
   // Updates the overall load progress and notifies the WebContents.
   void UpdateLoadProgress();
 
   // Returns this FrameTree's total load progress.
   double load_progress() { return load_progress_; }
@@ skipping 46 matching lines @@
 
   // Overall load progress.
   double load_progress_;
 
   DISALLOW_COPY_AND_ASSIGN(FrameTree);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_H_