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

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

Issue 7273038: ChromeOs: Call BrowserList::SessionEnding when shutting down via SIGTERM (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: " Created 9 years, 6 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_list.cc
diff --git a/chrome/browser/ui/browser_list.cc b/chrome/browser/ui/browser_list.cc
index cc536f1611d7e51c4c8f9d4100440910231c54ec..d7c9f6a9e69e64cf722c83c4b5df4c4bb50bba29 100644
--- a/chrome/browser/ui/browser_list.cc
+++ b/chrome/browser/ui/browser_list.cc
@@ -281,8 +281,8 @@ void BrowserList::AttemptExitInternal() {
// static
void BrowserList::NotifyAndTerminate(bool fast_path) {
#if defined(OS_CHROMEOS)
- if (!signout) return;
- NotifyWindowManagerAboutSignout();
+ if (!signout)
+ return;
#endif
if (fast_path) {
@@ -292,6 +292,7 @@ void BrowserList::NotifyAndTerminate(bool fast_path) {
}
#if defined(OS_CHROMEOS)
+ NotifyWindowManagerAboutSignout();
chromeos::CrosLibrary* cros_library = chromeos::CrosLibrary::Get();
if (cros_library->EnsureLoaded()) {
// If update has been installed, reboot, otherwise, sign out.
@@ -436,12 +437,11 @@ void BrowserList::AttemptUserExit() {
state->GetString(prefs::kApplicationLocale) != owner_locale &&
!state->IsManagedPreference(prefs::kApplicationLocale)) {
state->SetString(prefs::kApplicationLocale, owner_locale);
- state->ScheduleSavePersistentPrefs();
+ state->SavePersistentPrefs();
}
}
- if (FastShutdown()) {
+ if (FastShutdown())
return;
- }
#else
// Reset the restart bit that might have been set in cancelled restart
// request.
@@ -481,7 +481,7 @@ void BrowserList::AttemptExit() {
// static
void BrowserList::ExitCleanly() {
// We always mark exit cleanly.
- MarkAsCleanShutdown();
+ g_browser_process->EndSession();
AttemptExitInternal();
}
#endif
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698