Merge 32264 - Launch processes asynchronously so as not to block the UI thread. For now, renderer only, I'll take care of plugin/worker/utility processes in a followup change. (relanding 32203)

BUG=6844,27935
Review URL: http://codereview.chromium.org/397031

TBR=jam@chromium.org
Review URL: http://codereview.chromium.org/408011

git-svn-id: svn://svn.chromium.org/chrome/branches/249/src@32388 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 11 insertions, 5 deletions

diff --git a/chrome/browser/extensions/extension_host.cc b/chrome/browser/extensions/extension_host.cc
index ba7d54b..28f7403 100644
--- a/chrome/browser/extensions/extension_host.cc
+++ b/chrome/browser/extensions/extension_host.cc
@@ -136,6 +136,11 @@ ExtensionHost::ExtensionHost(Extension* extension, SiteInstance* site_instance,
   registrar_.Add(this, NotificationType::EXTENSION_HOST_VIEW_SHOULD_CLOSE,
                  Source<Profile>(profile_));
 #endif
+
+  // Listen for when the render process' handle is available so we can add it
+  // to the task manager then.
+  registrar_.Add(this, NotificationType::RENDERER_PROCESS_CREATED,
+                 Source<RenderProcessHost>(render_process_host()));
 }
 
 ExtensionHost::~ExtensionHost() {
@@ -188,11 +193,6 @@ void ExtensionHost::CreateRenderViewNow() {
   render_view_host_->CreateRenderView();
   NavigateToURL(url_);
   DCHECK(IsRenderViewLive());
-  LOG(INFO) << "Sending EXTENSION_PROCESS_CREATED";
-  NotificationService::current()->Notify(
-      NotificationType::EXTENSION_PROCESS_CREATED,
-      Source<Profile>(profile_),
-      Details<ExtensionHost>(this));
 }
 
 void ExtensionHost::NavigateToURL(const GURL& url) {
@@ -238,6 +238,12 @@ void ExtensionHost::Observe(NotificationType type,
 
     DismissPopup();
 #endif
+  } else if (type == NotificationType::RENDERER_PROCESS_CREATED) {
+    LOG(INFO) << "Sending EXTENSION_PROCESS_CREATED";
+    NotificationService::current()->Notify(
+        NotificationType::EXTENSION_PROCESS_CREATED,
+        Source<Profile>(profile_),
+        Details<ExtensionHost>(this));
   } else {
     NOTREACHED();
   }