Tie extension/app initialization to RenderView creation, not RenderViewHost creation

content/browser/renderer_host/render_view_host.cc

 // Copyright (c) 2011 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/renderer_host/render_view_host.h"
 
 #include <string>
 #include <utility>
 #include <vector>
 
@@ skipping 174 matching lines @@
   params.view_id = routing_id();
   params.session_storage_namespace_id = session_storage_namespace_->id();
   params.frame_name = frame_name;
   Send(new ViewMsg_New(params));
 
   // If it's enabled, tell the renderer to set up the Javascript bindings for
   // sending messages back to the browser.
   Send(new ViewMsg_AllowBindings(routing_id(), enabled_bindings_));
   // Let our delegate know that we created a RenderView.
   delegate_->RenderViewCreated(this);
+  content::GetContentClient()->browser()->RenderViewCreated(this);

[jam, 2011/08/11 18:13:37]

It's best to avoid adding methods to ContentBrowserClient unless there's no
other way (i.e. at the point that content needs to tell the embedder, there's no
other context). The reason is that ContentBrowserClient is already pretty large,
and the smaller we can keep it the easier it'll be for embedders.

In this case, we already tell the RenderViewHostObservers. Chrome implements one
in chrome/browser/renderer_host/chrome_render_view_host_observer.cc, so how
about moving the code there?

 
   FOR_EACH_OBSERVER(
       RenderViewHostObserver, observers_, RenderViewHostInitialized());
 
   return true;
 }
 
 bool RenderViewHost::IsRenderViewLive() const {
   return process()->HasConnection() && renderer_initialized_;
 }
@@ skipping 1129 matching lines @@
   if (view) {
     view->ShowPopupMenu(params.bounds,
                         params.item_height,
                         params.item_font_size,
                         params.selected_item,
                         params.popup_items,
                         params.right_aligned);
   }
 }
 #endif