Tweak commit interval for ScheduleSavePersistentPrefs.

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..26a0bc0a6555d6c43d0c71790dc9540b6bfa3ced 100644
--- a/chrome/common/important_file_writer.cc
+++ b/chrome/common/important_file_writer.cc
@@ -21,7 +21,11 @@ using base::TimeDelta;
 
 namespace {
 
-const int kDefaultCommitIntervalMs = 10000;
+#ifdef NDEBUG

[Paweł Hajdan Jr., 2011/03/18 19:59:03]

I'm not a fan of different behavior in debug vs. release builds. Could you
remove this #if?

[Denis Lagno, 2011/03/21 01:18:53]

I removed dependency on NDEBUG but instead added dependency on OFFICIAL_BUILD or
GOOGLE_CHROME_BUILD

+const int kDefaultCommitIntervalMs = 2000;

[Paweł Hajdan Jr., 2011/03/18 19:59:03]

I'm not sure whether decreasing this interval is the right solution to the
problem. I think we should be either committing everything immediately, or keep
the existing interval.

Another idea would be to make ScheduleWrite call DoScheduledWrite immediately.

Note: even if we post the write task with no delay, it's still executed on
separate thread. If the browser is going to be killed, that doesn't really
"solve" the "data is not saved" problem.

[brettw, 2011/03/18 20:30:21]

If we do keep this different, there should be a comment explaining why.

[Denis Lagno, 2011/03/21 01:18:53]

Done.

[Denis Lagno, 2011/03/21 01:18:53]

original problem I've encountered was the fact that chromeos::RestartJob call
fails to commit pending writes for PrefService.  This problem was resolved by
enforcing committing on this exit path.

I like current implementation with delayed writes.  We keep a lot of stuff in
PrefServices and I am quite sure that there are bursts of writes to them so
commit interval is good optimization.

However I believe that optimal value of commit interval for stable version of
Chrome does not exceed 1-2 seconds.  Values higher than 1-2 seconds give
near-zero gain in performance but they give increasing unsafety.  Probability of
occurence of some unrelated crash grows fast with time passed.  User can do a
real lot in 10 seconds:)

+#else
+const int kDefaultCommitIntervalMs = 100000;
+#endif
 
 class WriteToDiskTask : public Task {
  public: