Introduce a new framework for back-and-forth tracked/protected preferences migration.

base/files/important_file_writer.h

Index: base/files/important_file_writer.h
diff --git a/base/files/important_file_writer.h b/base/files/important_file_writer.h
index ba1c745a4efd9fd36e4cf44b2f884c3c6c53b7b4..1705d051c47cbf900c980bca90c2b551224faa79 100644
--- a/base/files/important_file_writer.h
+++ b/base/files/important_file_writer.h
@@ -9,6 +9,7 @@
 
 #include "base/base_export.h"
 #include "base/basictypes.h"
+#include "base/callback.h"
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/threading/non_thread_safe.h"
@@ -89,6 +90,13 @@ class BASE_EXPORT ImportantFileWriter : public NonThreadSafe {
   // Serialize data pending to be saved and execute write on backend thread.
   void DoScheduledWrite();
 
+  // Registers |on_next_successful_write| to be called once, on the next
+  // successful write event. Only one callback should be set at once, if this

[Nico, 2014/05/06 21:14:50]

"…can be set at once." (remove comment after that)

[gab, 2014/05/07 03:59:09]

Done.

+  // isn't sufficient in the future, this chould be expanded into an observer
+  // API.
+  void RegisterOnNextSuccessfulWriteCallback(
+      const base::Closure& on_next_successful_write);
+
   TimeDelta commit_interval() const {
     return commit_interval_;
   }
@@ -98,6 +106,13 @@ class BASE_EXPORT ImportantFileWriter : public NonThreadSafe {
   }
 
  private:
+  // If |result| is true and |on_next_successful_write_| is set, invokes
+  // |on_successful_write_| and then resets it; no-ops otherwise.
+  void ForwardSuccessfulWrite(bool result);
+
+  // Invoked once and then reset on the next successful write event.
+  base::Closure on_next_successful_write_;
+
   // Path being written to.
   const FilePath path_;
 
@@ -113,6 +128,8 @@ class BASE_EXPORT ImportantFileWriter : public NonThreadSafe {
   // Time delta after which scheduled data will be written to disk.
   TimeDelta commit_interval_;
 
+  WeakPtrFactory<ImportantFileWriter> weak_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(ImportantFileWriter);
 };