Remove chrome::NOTIFICATION_PROFILE_DESTROYED usage from src/extensions

chrome/browser/extensions/chrome_process_manager_delegate.cc

Index: chrome/browser/extensions/chrome_process_manager_delegate.cc
diff --git a/chrome/browser/extensions/chrome_process_manager_delegate.cc b/chrome/browser/extensions/chrome_process_manager_delegate.cc
index 12e0c161d669aad0f829cb830bcb271590cc6249..50dc2841c7768b598b3980f1a936aaf7f4ca221d 100644
--- a/chrome/browser/extensions/chrome_process_manager_delegate.cc
+++ b/chrome/browser/extensions/chrome_process_manager_delegate.cc
@@ -30,8 +30,9 @@ ChromeProcessManagerDelegate::ChromeProcessManagerDelegate() {
   registrar_.Add(this,
                  chrome::NOTIFICATION_PROFILE_CREATED,
                  content::NotificationService::AllSources());
-  // TODO(jamescook): Move observation of NOTIFICATION_PROFILE_DESTROYED here.
-  // http://crbug.com/392658
+  registrar_.Add(this,
+                 chrome::NOTIFICATION_PROFILE_DESTROYED,
+                 content::NotificationService::AllSources());
 }
 
 ChromeProcessManagerDelegate::~ChromeProcessManagerDelegate() {
@@ -85,7 +86,11 @@ void ChromeProcessManagerDelegate::Observe(
       OnProfileCreated(profile);
       break;
     }
-
+    case chrome::NOTIFICATION_PROFILE_DESTROYED: {
+      Profile* profile = content::Source<Profile>(source).ptr();
+      OnProfileDestroyed(profile);
+      break;
+    }
     default:
       NOTREACHED();
   }
@@ -140,4 +145,24 @@ void ChromeProcessManagerDelegate::OnProfileCreated(Profile* profile) {
   manager->MaybeCreateStartupBackgroundHosts();
 }
 
+void ChromeProcessManagerDelegate::OnProfileDestroyed(Profile* profile) {
+  // Close background hosts when the last profile is closed so that they
+  // have time to shutdown various objects on different threads. The
+  // ProfileManager destructor is called too late in the shutdown sequence.
+  // http://crbug.com/15708
+  ProcessManager* manager = ExtensionSystem::Get(profile)->process_manager();
+  if (manager)
+    manager->CloseBackgroundHosts();
+
+  // If this profile owns an incognito profile then close the incognito

[James Cook, 2014/07/24 18:27:54]

This mimics the behavior of the existing code (specifically the part where an
incognito process manager was listening for PROFILE_DESTROYED for the original
non-incognito context.)

It's unclear to me why it's needed. I guess it's because a non-incognito profile
could be destroyed *before* the incognito profile that it owns? So we need to
shut down split-mode extensions before the incognito profile closes?

[Yoyo Zhou, 2014/07/24 20:25:01]

From the CL that added this, I wrote

(which happens in a few tests)

so I think it only happens in test code. That can go into the comment here.

[James Cook, 2014/07/24 21:02:35]

Done.

+  // background hosts as well. http://crbug.com/138843
+  if (!profile->IsOffTheRecord() && profile->HasOffTheRecordProfile()) {
+    ProcessManager* incognito_manager =
+        ExtensionSystem::Get(profile->GetOffTheRecordProfile())
+            ->process_manager();
+    if (incognito_manager)
+      incognito_manager->CloseBackgroundHosts();
+  }
+}
+
 }  // namespace extensions