Provide lazy CommitPendingWrites in addition to eager SavePersistentPrefs.

chrome/common/important_file_writer.cc

Index: chrome/common/important_file_writer.cc
diff --git a/chrome/common/important_file_writer.cc b/chrome/common/important_file_writer.cc
index f5c5fc115c1dcee81bc86f71d7c32235b8264c76..e770678aeb6544c7ff5396a8e51dca1be36f95c1 100644
--- a/chrome/common/important_file_writer.cc
+++ b/chrome/common/important_file_writer.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
@@ -106,8 +106,8 @@ void ImportantFileWriter::WriteNow(const std::string& data) {
   if (HasPendingWrite())
     timer_.Stop();
 
-  if (!file_message_loop_proxy_->PostTask(FROM_HERE,
-                                          new WriteToDiskTask(path_, data))) {
+  if (!file_message_loop_proxy_->PostTask(
+      FROM_HERE, new WriteToDiskTask(path_, data))) {

[Mattias Nissler (ping if slow), 2011/03/25 15:29:57]

any reason for the formatting change here? I found the previous version easier
to read.

[Denis Lagno, 2011/03/25 15:41:22]

In old versions of CL there was argument added, now it is gone so it remained in
CL by mistake.

However at least one reason to prefer indentation on the right is that if you
ever need to rename some identifiers then indentation on the right will just
remain correct while indentation on the left needs to be reformatted.

     // Posting the task to background message loop is not expected
     // to fail, but if it does, avoid losing data and just hit the disk
     // on the current thread.