Move preference MACs to the protected preference stores.

chrome/browser/prefs/pref_hash_store.h

Index: chrome/browser/prefs/pref_hash_store.h
diff --git a/chrome/browser/prefs/pref_hash_store.h b/chrome/browser/prefs/pref_hash_store.h
index a6d97277b70b61acf57a3e3a9d64dd79bd2be7db..ed6ef17530fec11ddfd148aa8cd5dbd4769b6f80 100644
--- a/chrome/browser/prefs/pref_hash_store.h
+++ b/chrome/browser/prefs/pref_hash_store.h
@@ -7,21 +7,21 @@
 
 #include "base/memory/scoped_ptr.h"
 
+class HashStoreContents;
 class PrefHashStoreTransaction;
 
-// Stores hashes of and verifies preference values via
-// PrefHashStoreTransactions.
+// Holds the configuration and implementation used to calculate and verify
+// preference MACs.
 class PrefHashStore {

[gab, 2014/06/17 02:00:05]

Add a TODO as such:

// TODO(gab): Rename this class to PrefHashTransactionManager as it is no longer
even used as a store.

[erikwright (departed), 2014/06/17 19:07:23]

Done.

  public:
   virtual ~PrefHashStore() {}
 
   // Returns a PrefHashStoreTransaction which can be used to perform a series
-  // of checks/transformations on the hash store.
-  virtual scoped_ptr<PrefHashStoreTransaction> BeginTransaction() = 0;
-
-  // Commits this store to disk if it has changed since the last call to this
-  // method.
-  virtual void CommitPendingWrite() = 0;
+  // of operations on the hash store. |storage| MAY be used as the backing store
+  // depending on the implementation. Therefore the HashStoreContents used for
+  // related transactions should correspond to the same underlying data store.
+  virtual scoped_ptr<PrefHashStoreTransaction> BeginTransaction(
+      scoped_ptr<HashStoreContents> storage) = 0;
 };
 
 #endif  // CHROME_BROWSER_PREFS_PREF_HASH_STORE_H_