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

chrome/browser/metrics/metrics_service.h

Index: chrome/browser/metrics/metrics_service.h
diff --git a/chrome/browser/metrics/metrics_service.h b/chrome/browser/metrics/metrics_service.h
index 5a031edf9257a48c389e56e630b9bc3e729ca0c9..6ad29cb4041c7d8099847af27349816b42ac7800 100644
--- a/chrome/browser/metrics/metrics_service.h
+++ b/chrome/browser/metrics/metrics_service.h
@@ -122,6 +122,9 @@ class MetricsService : public NotificationObserver,
   // at shutdown, but we can do it as we reduce the list as well.
   void StoreUnsentLogs();
 
+  // Records that the browser was shut down cleanly.
+  void LogCleanShutdown();

[jar (doing other things), 2011/06/29 04:00:09]

Can you motivate better why we want to expose this interface publicly?  It seems
to shout out to invite callers to call it, and claim we exited cleanly (which it
doesn't even do!)

+
 #if defined(OS_CHROMEOS)
   // Start the external metrics service, which collects metrics from Chrome OS
   // and passes them to UMA.
@@ -286,9 +289,6 @@ class MetricsService : public NotificationObserver,
   // Records a renderer process hang.
   void LogRendererHang();
 
-  // Records that the browser was shut down cleanly.
-  void LogCleanShutdown();
-
   // Set the value in preferences for the number of bookmarks and folders
   // in node. The pref key for the number of bookmarks in num_bookmarks_key and
   // the pref key for number of folders in num_folders_key.