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

Unified Diff: chrome/browser/ui/browser_list.h

Issue 10409022: Move application lifetime functionality off BrowserList. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 7 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 | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/browser_list.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_list.h
===================================================================
--- chrome/browser/ui/browser_list.h (revision 137868)
+++ chrome/browser/ui/browser_list.h (working copy)
@@ -64,69 +64,9 @@
// WARNING #2: this will always be NULL in unit tests run on the bots.
static Browser* GetLastActive();
- // Checks if the browser can be automatically restarted to install upgrades
- // The browser can be automatically restarted when:
- // 1. It's in the background mode (no visible windows).
- // 2. An update exe is present in the install folder.
- static bool CanRestartForUpdate();
-
- // Starts a user initiated exit process. Called from Browser::Exit.
- // On platforms other than ChromeOS, this is equivalent to
- // CloseAllBrowsers() On ChromeOS, this tells session manager
- // that chrome is signing out, which lets session manager send
- // SIGTERM to start actual exit process.
- static void AttemptUserExit();
-
- // Starts a user initiated restart process. On platforms other than
- // chromeos, this sets a restart bit in the preference so that
- // chrome will be restarted at the end of shutdown process. On
- // ChromeOS, this simply exits the chrome, which lets sesssion
- // manager re-launch the browser with restore last session flag.
- static void AttemptRestart();
-
- // Attempt to exit by closing all browsers. This is equivalent to
- // CloseAllBrowsers() on platforms where the application exits
- // when no more windows are remaining. On other platforms (the Mac),
- // this will additionally exit the application if all browsers are
- // successfully closed.
- // Note that he exit process may be interrupted by download or
- // unload handler, and the browser may or may not exit.
- static void AttemptExit();
-
-#if defined(OS_CHROMEOS)
- // This is equivalent to AttemptUserExit, except that it always set
- // exit cleanly bit. ChromeOS checks if it can exit without user
- // interactions, so it will always exit the browser. This is used to
- // handle SIGTERM on chromeos which is a signal to force shutdown
- // the chrome.
- static void ExitCleanly();
-#endif
-
- // Closes all browsers. If the session is ending the windows are closed
- // directly. Otherwise the windows are closed by way of posting a WM_CLOSE
- // message.
- static void CloseAllBrowsers();
-
// Closes all browsers for |profile|.
static void CloseAllBrowsersWithProfile(Profile* profile);
- // Begins shutdown of the application when the desktop session is ending.
- static void SessionEnding();
-
- // Tells the BrowserList to keep the application alive after the last Browser
- // closes. This is implemented as a count, so callers should pair their calls
- // to StartKeepAlive() with matching calls to EndKeepAlive() when they no
- // longer need to keep the application running.
- static void StartKeepAlive();
-
- // Stops keeping the application alive after the last Browser is closed.
- // Should match a previous call to StartKeepAlive().
- static void EndKeepAlive();
-
- // Returns true if application will continue running after the last Browser
- // closes.
- static bool WillKeepAlive();
-
// Browsers are added to the list before they have constructed windows,
// so the |window()| member function may return NULL.
static const_iterator begin();
@@ -148,26 +88,9 @@
// Returns true if at least one incognito session is active for |profile|.
static bool IsOffTheRecordSessionActiveForProfile(Profile* profile);
- // Send out notifications.
- // For ChromeOS, also request session manager to end the session.
- static void NotifyAndTerminate(bool fast_path);
-
- // Called once the application is exiting.
- static void OnAppExiting();
-
- // Called once the application is exiting to do any platform specific
- // processing required.
- static void HandleAppExitingForPlatform();
-
private:
// Helper method to remove a browser instance from a list of browsers
static void RemoveBrowserFrom(Browser* browser, BrowserVector* browser_list);
- static void MarkAsCleanShutdown();
- static void AttemptExitInternal();
-
- // Counter of calls to StartKeepAlive(). If non-zero, the application will
- // continue running after the last browser has exited.
- static int keep_alive_count_;
};
class TabContentsWrapper;
« no previous file with comments | « chrome/browser/ui/browser.cc ('k') | chrome/browser/ui/browser_list.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698