Start using RenderFrameProxyHost objects.

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/cross_process_frame_connector.h"
+#include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/frame_host/frame_tree_node.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/frame_host/render_widget_host_view_child_frame.h"
+#include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 #include "content/browser/site_instance_impl.h"
 #include "content/common/frame_messages.h"
 #include "ipc/ipc_message.h"
 
 namespace content {
 
 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);
 
   if (!frame_tree_node_->IsMainFrame() &&
       frame_tree_node_->parent()
               ->render_manager()
               ->current_frame_host()
               ->GetSiteInstance() == site_instance) {
     // The RenderFrameHost navigating cross-process is destroyed and a proxy for
     // it is created in the parent's process. CrossProcessFrameConnector
     // initialization only needs to happen on an initial cross-process
     // navigation, when the RenderFrameHost leaves the same process as its
     // parent. The same CrossProcessFrameConnector is used for subsequent cross-
     // process navigations, but it will be destroyed if the frame is
     // navigated back to the same SiteInstance as its parent.
     cross_process_frame_connector_.reset(new CrossProcessFrameConnector(this));
   }
+
+  RenderFrameHostManager* manager = NULL;
+  if (frame_tree_node_->parent()) {
+    manager = frame_tree_node_->parent()->render_manager();
+  }
+
+  LOG(ERROR) << "RFPH::RFPH[" << this << "]:"
+    " process:" << site_instance_->GetProcess()->GetID() <<
+    ", routing:" << routing_id_ <<
+    ", site:" << site_instance_->GetSiteURL() <<
+    ", parent: " << manager;
+
 }
 
 RenderFrameProxyHost::~RenderFrameProxyHost() {
   if (GetProcess()->HasConnection())
     Send(new FrameMsg_DeleteProxy(routing_id_));
 
+  LOG(ERROR) << "RFPH::~RFPH[" << this << "]: "
+      << site_instance_->GetSiteURL();
+
   GetProcess()->RemoveRoute(routing_id_);
 }
 
 void RenderFrameProxyHost::SetChildRWHView(RenderWidgetHostView* view) {
   cross_process_frame_connector_->set_view(
       static_cast<RenderWidgetHostViewChildFrame*>(view));
 }
 
 RenderViewHostImpl* RenderFrameProxyHost::GetRenderViewHost() {
   if (render_frame_host_.get())
@@ skipping 20 matching lines @@
     return true;
 
   // TODO(nasko): This can be removed once we don't have a swapped out state on
   // RenderFrameHosts. See https://crbug.com/357747.
   if (render_frame_host_.get())
     return render_frame_host_->OnMessageReceived(msg);
 
   return false;
 }
 
+bool RenderFrameProxyHost::InitRenderFrameProxy() {
+  LOG(ERROR) << "RFPH::Init[" << this << "]:";
+  // The process may (if we're sharing a process with another host that already
+  // initialized it) or may not (we have our own process or the old process
+  // crashed) have been initialized. Calling Init multiple times will be
+  // ignored, so this is safe.
+  if (!site_instance_->GetProcess()->Init()) {
+    LOG(ERROR) << "RFPH::Init[" << this << "]:"
+        << " failed to init process";
+    return false;
+  }
+
+  DCHECK(GetProcess()->HasConnection());
+  DCHECK(GetProcess()->GetBrowserContext());
+
+  int parent_routing_id = MSG_ROUTING_NONE;
+  if (frame_tree_node_->parent()) {
+    parent_routing_id = frame_tree_node_->parent()->render_manager()->

[ncarter (slow), 2014/06/25 01:07:56]

I wonder if |parent_routing_id| should/could be a const member set in the ctor
(am worried about weird orderings here where this is called in the middle of a
parent change .... ??!) -- this just looks like something that Would Be Bad if
it were to change dynamically.

+        GetRoutingIdForSiteInstance(site_instance_);
+    LOG(ERROR) << "RFPH::Init[" << this << "]:"
+        << " parent:" << parent_routing_id;
+    CHECK_NE(parent_routing_id, MSG_ROUTING_NONE);
+  }
+
+  Send(new FrameMsg_NewFrameProxy(
+      routing_id_,
+      parent_routing_id,
+      frame_tree_node_->frame_tree()->GetRenderViewHost(
+          site_instance_)->GetRoutingID()));
+
+  return true;
+}
+
 }  // namespace content