diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-18 00:03:23 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-18 00:03:23 +0000 |
commit | fdb7ee3dc832589e4ee911a598d4f7fd1054e551 (patch) | |
tree | 1f436470c99e3d4f861252de87f686d8048e841b /chrome/browser/extensions/extension_host.cc | |
parent | 7d8dc6258b7c9dd771eaf509f5403b602a838388 (diff) | |
download | chromium_src-fdb7ee3dc832589e4ee911a598d4f7fd1054e551.zip chromium_src-fdb7ee3dc832589e4ee911a598d4f7fd1054e551.tar.gz chromium_src-fdb7ee3dc832589e4ee911a598d4f7fd1054e551.tar.bz2 |
Revert 32203,32204,32205 - 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.
BUG=6844
Review URL: http://codereview.chromium.org/397002
Review URL: http://codereview.chromium.org/402033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32238 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/extension_host.cc')
-rw-r--r-- | chrome/browser/extensions/extension_host.cc | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/chrome/browser/extensions/extension_host.cc b/chrome/browser/extensions/extension_host.cc index b0771ad..85f48de5 100644 --- a/chrome/browser/extensions/extension_host.cc +++ b/chrome/browser/extensions/extension_host.cc @@ -124,11 +124,6 @@ ExtensionHost::ExtensionHost(Extension* extension, SiteInstance* site_instance, render_view_host_->AllowBindings(BindingsPolicy::EXTENSION); if (enable_dom_automation_) render_view_host_->AllowBindings(BindingsPolicy::DOM_AUTOMATION); - - // 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() { @@ -180,6 +175,11 @@ void ExtensionHost::CreateRenderViewNow() { render_view_host_->CreateRenderView(profile_->GetRequestContext()); 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) { @@ -216,12 +216,6 @@ void ExtensionHost::Observe(NotificationType type, NavigateToURL(url_); } else if (type == NotificationType::BROWSER_THEME_CHANGED) { InsertThemeCSS(); - } 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(); } |