diff options
author | atwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-26 17:46:27 +0000 |
---|---|---|
committer | atwilson@chromium.org <atwilson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-26 17:46:27 +0000 |
commit | 7998899b24cd98227e676dc858e061e9557794df (patch) | |
tree | 289d468677b3f62d0b8cc427985907c2f347fe6e /chrome/browser/browser_list.cc | |
parent | cffd78950f9e8e740085dbec9ecec7472e6eb98d (diff) | |
download | chromium_src-7998899b24cd98227e676dc858e061e9557794df.zip chromium_src-7998899b24cd98227e676dc858e061e9557794df.tar.gz chromium_src-7998899b24cd98227e676dc858e061e9557794df.tar.bz2 |
Added clarifying comment to BrowserList::RemoveBrowser().
Added comment describing why we don't need to call ShutdownSessionServices()
BUG=53360
TEST=None needed (comment change only)
Review URL: http://codereview.chromium.org/3186034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57545 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_list.cc')
-rw-r--r-- | chrome/browser/browser_list.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/chrome/browser/browser_list.cc b/chrome/browser/browser_list.cc index ed0991a..0ce399d 100644 --- a/chrome/browser/browser_list.cc +++ b/chrome/browser/browser_list.cc @@ -201,7 +201,10 @@ void BrowserList::RemoveBrowser(Browser* browser) { (browser_shutdown::IsTryingToQuit() || g_browser_process->IsShuttingDown())) { // Last browser has just closed, and this is a user-initiated quit or there - // is no module keeping the app alive, so send out our notification. + // is no module keeping the app alive, so send out our notification. No need + // to call ProfileManager::ShutdownSessionServices() as part of the + // shutdown, because Browser::WindowClosing() already makes sure that the + // SessionService is created and notified. NotificationService::current()->Notify(NotificationType::APP_TERMINATING, NotificationService::AllSources(), NotificationService::NoDetails()); |