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

Unified Diff: chrome/browser/sessions/app_restore_service.h

Issue 10875027: Restart running apps when chrome restarts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: typo Created 8 years, 4 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
Index: chrome/browser/sessions/app_restore_service.h
diff --git a/chrome/browser/sessions/app_restore_service.h b/chrome/browser/sessions/app_restore_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..777929c9d64b49023e6b8459edccba9b22e3ca76
--- /dev/null
+++ b/chrome/browser/sessions/app_restore_service.h
@@ -0,0 +1,53 @@
+// Copyright (c) 2012 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.
+
+#ifndef CHROME_BROWSER_SESSIONS_APP_RESTORE_SERVICE_H_
+#define CHROME_BROWSER_SESSIONS_APP_RESTORE_SERVICE_H_
+
+#include "chrome/browser/profiles/profile_keyed_service.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+
+#include <list>
+#include <string>
+#include <vector>
+
+class PrefService;
+class Profile;
+
+namespace extensions {
+class ExtensionHost;
+}
+
+// Tracks what apps need to be restarted when the browser restarts.
Mihai Parparita -not on Chrome 2012/09/04 04:14:48 Does it make sense for this to be in chrome/browse
koz (OOO until 15th September) 2012/09/04 07:10:20 Done.
+class AppRestoreService : public ProfileKeyedService,
+ public content::NotificationObserver {
+ public:
+ explicit AppRestoreService(Profile* profile);
+
+ // Dispatches the app.runtime.onRestarted event to apps that were running in
+ // the last session.
+ void RestoreApps();
+
+ // Dispatches the app.runtime.onRestarted event to the apps identified by the
+ // extension ids in |apps|.
+ void RestartApps(std::list<std::string>& apps);
Mihai Parparita -not on Chrome 2012/09/04 04:14:48 This appears to be unused/undefined.
koz (OOO until 15th September) 2012/09/04 07:10:20 Done.
+
+ // content::NotificationObserver.
+ virtual void Observe(int type,
Mihai Parparita -not on Chrome 2012/09/04 04:14:48 Needs OVERRIDE (also, it could be private).
koz (OOO until 15th September) 2012/09/04 07:10:20 Done.
+ const content::NotificationSource& source,
+ const content::NotificationDetails& details);
+
+ static void RegisterUserPrefs(PrefService* prefService);
+
+ private:
+ void RecordAppStart(const std::string& extension_id);
+ void RecordAppStop(const std::string& extension_id);
+ void RestoreApp(const std::string& extension_id);
+
+ content::NotificationRegistrar registrar_;
+ Profile* profile_;
+};
+
+#endif // CHROME_BROWSER_SESSIONS_APP_RESTORE_SERVICE_H_

Powered by Google App Engine
This is Rietveld 408576698