Don't commit prefs::kStabilitySessionEndCompleted change immediately on chromeos

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index a48c7e2a46bf8ce4e0410ecaf4b25ef5362bec0d..28cecddd97a4908440fc7992c279df101de81731 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -362,13 +362,10 @@ void BrowserProcessImpl::EndSession() {
 
   // Tell the metrics service it was cleanly shutdown.
   MetricsService* metrics = g_browser_process->metrics_service();
-  if (metrics && local_state()) {
+  // MetricsService::RecordStartOfSessionEnd writes to prefs immediately.

[jar (doing other things), 2013/10/02 01:18:31]

Can you point to where this happens?  I don't see it.

If it is true, then the comment should be between lines 366 and 367, and hence
inside curlies.

+  if (metrics && local_state())
     metrics->RecordStartOfSessionEnd();
 
-    // MetricsService lazily writes to prefs, force it to write now.
-    local_state()->CommitPendingWrite();
-  }
-
   // http://crbug.com/125207
   base::ThreadRestrictions::ScopedAllowWait allow_wait;