Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Unified Diff: apps/app_lifetime_monitor.cc

Issue 2623273011: Move Chrome terminating notification out of //apps (Closed)
Patch Set: remove DEPS rule Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « apps/app_lifetime_monitor.h ('k') | apps/app_restore_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: apps/app_lifetime_monitor.cc
diff --git a/apps/app_lifetime_monitor.cc b/apps/app_lifetime_monitor.cc
index 637c90916cdbbdbdb07d8056234d4b030913528d..1edaa3b469fba791f4a45675b3d94d219cce8dfa 100644
--- a/apps/app_lifetime_monitor.cc
+++ b/apps/app_lifetime_monitor.cc
@@ -4,7 +4,6 @@
#include "apps/app_lifetime_monitor.h"
-#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/profiles/profile.h"
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_service.h"
@@ -28,9 +27,6 @@ AppLifetimeMonitor::AppLifetimeMonitor(Profile* profile)
registrar_.Add(this,
extensions::NOTIFICATION_EXTENSION_HOST_DESTROYED,
content::NotificationService::AllSources());
- registrar_.Add(
- this, chrome::NOTIFICATION_APP_TERMINATING,
- content::NotificationService::AllSources());
AppWindowRegistry* app_window_registry =
AppWindowRegistry::Factory::GetForBrowserContext(profile_,
@@ -72,11 +68,6 @@ void AppLifetimeMonitor::Observe(int type,
NotifyAppStop(extension->id());
break;
}
-
- case chrome::NOTIFICATION_APP_TERMINATING: {
- NotifyChromeTerminating();
- break;
- }
}
}
@@ -144,9 +135,4 @@ void AppLifetimeMonitor::NotifyAppStop(const std::string& app_id) {
observer.OnAppStop(profile_, app_id);
}
-void AppLifetimeMonitor::NotifyChromeTerminating() {
- for (auto& observer : observers_)
- observer.OnChromeTerminating();
-}
-
} // namespace apps
« no previous file with comments | « apps/app_lifetime_monitor.h ('k') | apps/app_restore_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698