Cleanup nits in ImportantFileWriter and JsonPrefStore.

components/bookmarks/browser/bookmark_storage.cc

Index: components/bookmarks/browser/bookmark_storage.cc
diff --git a/components/bookmarks/browser/bookmark_storage.cc b/components/bookmarks/browser/bookmark_storage.cc
index bc3fbfb24cc91827f118a4abfb6e9daab8bad259..ec053a7163c8c28e743619038f1ea1b9b631c800 100644
--- a/components/bookmarks/browser/bookmark_storage.cc
+++ b/components/bookmarks/browser/bookmark_storage.cc
@@ -143,10 +143,11 @@ BookmarkStorage::BookmarkStorage(
     const base::FilePath& profile_path,
     base::SequencedTaskRunner* sequenced_task_runner)
     : model_(model),
-      writer_(profile_path.Append(kBookmarksFileName), sequenced_task_runner),
+      writer_(profile_path.Append(kBookmarksFileName),
+              sequenced_task_runner,
+              base::TimeDelta::FromMilliseconds(kSaveDelayMS)),
+      sequenced_task_runner_(sequenced_task_runner),

[tfarina, 2015/08/05 21:09:55]

yay. looks like we missed the opportunity to initialize it in the initializer
list. thanks for fixing this.

       weak_factory_(this) {
-  sequenced_task_runner_ = sequenced_task_runner;
-  writer_.set_commit_interval(base::TimeDelta::FromMilliseconds(kSaveDelayMS));
   sequenced_task_runner_->PostTask(FROM_HERE,
                                    base::Bind(&BackupCallback, writer_.path()));
 }