ChromeOs: Call BrowserList::SessionEnding when shutting down via SIGTERM

chrome/browser/ui/browser_list.h

 // Copyright (c) 2011 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_UI_BROWSER_LIST_H_
 #define CHROME_BROWSER_UI_BROWSER_LIST_H_
 #pragma once
 
 #include <set>
 #include <vector>
@@ skipping 109 matching lines @@
 
   // 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. ChroemOS 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();
 
-  // Begins shutdown of the application when the desktop session is ending.
+  // Begins shutdown of the application when the desktop session is
+  // ending.  This first calls BrowserProcess::EndSession then force
+  // closes browsers, therefore any crashes after EndSession won't be
+  // included in UMA stats.
   static void SessionEnding();
 
   // Returns true if there is at least one Browser with the specified profile.
   static bool HasBrowserWithProfile(Profile* profile);
 
   // 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();
@@ skipping 40 matching lines @@
   // Send out notifications.
   // For ChromeOS, also request session manager to end the session.
   static void NotifyAndTerminate(bool fast_path);
 
   // Called once there are no more browsers open and the application is exiting.
   static void AllBrowsersClosedAndAppExiting();
 
  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 MarkAsCleanShutdownInUserProfiles();
   static void AttemptExitInternal();
 
   static BrowserVector browsers_;
   static BrowserVector last_active_browsers_;
   static ObserverList<Observer> observers_;
 
   // Counter of calls to StartKeepAlive(). If non-zero, the application will
   // continue running after the last browser has exited.
   static int keep_alive_count_;
 };
@@ skipping 63 matching lines @@
   // iterator over the TabContentsWrappers doing background printing.
   std::set<TabContentsWrapper*>::const_iterator bg_printing_iterator_;
 
   // Current TabContents, or NULL if we're at the end of the list. This can
   // be extracted given the browser iterator and index, but it's nice to cache
   // this since the caller may access the current host many times.
   TabContentsWrapper* cur_;
 };
 
 #endif  // CHROME_BROWSER_UI_BROWSER_LIST_H_