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

Unified Diff: chrome/browser/lifetime/application_lifetime.h

Issue 1382563002: Split the desktop portion of application_lifetime.cc into its own file. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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 | « no previous file | chrome/browser/lifetime/application_lifetime.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/lifetime/application_lifetime.h
diff --git a/chrome/browser/lifetime/application_lifetime.h b/chrome/browser/lifetime/application_lifetime.h
index 3cf18549cd37768132f3af62cd9e806c18beded3..20e33e7f582c194a36975f68853abdf27b06020b 100644
--- a/chrome/browser/lifetime/application_lifetime.h
+++ b/chrome/browser/lifetime/application_lifetime.h
@@ -5,19 +5,15 @@
#ifndef CHROME_BROWSER_LIFETIME_APPLICATION_LIFETIME_H_
#define CHROME_BROWSER_LIFETIME_APPLICATION_LIFETIME_H_
-#include "base/compiler_specific.h"
+#include "base/basictypes.h"
-class Browser;
+// TODO(thestig): IYWU instead of relying on this.
+#if !defined(OS_ANDROID) && !defined(OS_IOS)
+#include "chrome/browser/lifetime/application_lifetime_desktop.h"
+#endif
namespace chrome {
-// 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.
-void AttemptUserExit();
-
// Starts to collect shutdown traces. On ChromeOS this will start immediately
// on AttemptUserExit() and all other systems will start once all tabs are
// closed.
@@ -30,26 +26,6 @@ void StartShutdownTracing();
// manager re-launch the browser with restore last session flag.
void AttemptRestart();
-#if defined(OS_WIN)
-enum AshExecutionStatus {
- ASH_KEEP_RUNNING,
- ASH_TERMINATE,
-};
-
-// Helper function to activate the desktop from Ash mode. The
-// |ash_execution_status| parameter indicates if we should exit Ash after
-// activating desktop.
-void ActivateDesktopHelper(AshExecutionStatus ash_execution_status);
-
-// Windows 7/8 specific: Like AttemptRestart but if chrome is running
-// in desktop mode it starts in metro mode and vice-versa. The switching like
-// the restarting is controlled by a preference.
-void AttemptRestartToDesktopMode();
-// Launches Chrome into Windows 8 metro mode on Windows 8. On Windows 7 it
-// launches Chrome into Windows ASH.
-void AttemptRestartToMetroMode();
-#endif
-
// 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),
@@ -59,44 +35,9 @@ void AttemptRestartToMetroMode();
// unload handler, and the browser may or may not exit.
void AttemptExit();
-#if defined(OS_CHROMEOS)
-// Shutdown chrome cleanly without blocking. This is called
-// when SIGTERM is received on Chrome OS, and always sets
-// exit-cleanly bit and exits the browser, even if there is
-// ongoing downloads or a page with onbeforeunload handler.
-//
-// If you need to exit or restart in your code on ChromeOS,
-// use AttemptExit or AttemptRestart respectively.
-void ExitCleanly();
-#endif
-
-// Closes all browsers and if successful, quits.
-void CloseAllBrowsersAndQuit();
-
-// 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. This will quit the application if there is nothing other than
-// browser windows keeping it alive or the application is quitting.
-void CloseAllBrowsers();
-
// Begins shutdown of the application when the desktop session is ending.
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 IncrementKeepAliveCount() with matching calls to DecrementKeepAliveCount()
-// when they no
-// longer need to keep the application running.
-void IncrementKeepAliveCount();
-
-// Stops keeping the application alive after the last Browser is closed.
-// Should match a previous call to IncrementKeepAliveCount().
-void DecrementKeepAliveCount();
-
-// Returns true if application will continue running after the last Browser
-// closes.
-bool WillKeepAlive();
-
// Emits APP_TERMINATING notification. It is guaranteed that the
// notification is sent only once.
void NotifyAppTerminating();
@@ -108,17 +49,6 @@ void NotifyAndTerminate(bool fast_path);
// Called once the application is exiting.
void OnAppExiting();
-// Called once the application is exiting to do any platform specific
-// processing required.
-void HandleAppExitingForPlatform();
-
-// Returns true if we can start the shutdown sequence for the browser, i.e. the
-// last browser window is being closed.
-bool ShouldStartShutdown(Browser* browser);
-
-// Disable browser shutdown for unit tests.
-void DisableShutdownForTesting(bool disable_shutdown_for_testing);
-
} // namespace chrome
#endif // CHROME_BROWSER_LIFETIME_APPLICATION_LIFETIME_H_
« no previous file with comments | « no previous file | chrome/browser/lifetime/application_lifetime.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698