Sanitize PrefStore interface.

chrome/common/json_pref_store.h

 // Copyright (c) 2010 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.
 
 #ifndef CHROME_COMMON_JSON_PREF_STORE_H_
 #define CHROME_COMMON_JSON_PREF_STORE_H_
 #pragma once
 
 #include <string>
 
+#include "base/basictypes.h"
+#include "base/file_path.h"
+#include "base/observer_list.h"
 #include "base/scoped_ptr.h"
-#include "chrome/common/pref_store.h"
+#include "chrome/common/persistent_pref_store.h"

[danno, 2010/12/08 13:08:45]

alpha

[Mattias Nissler (ping if slow), 2010/12/09 10:20:20]

Done.

 #include "chrome/common/important_file_writer.h"
 
 namespace base {
 class MessageLoopProxy;
 }
 
 class DictionaryValue;
 class FilePath;
+class Value;
 
-class JsonPrefStore : public PrefStore,
+// A writable PrefStore implementation that is used for user preferences.
+class JsonPrefStore : public PersistentPrefStore,
                       public ImportantFileWriter::DataSerializer {
  public:
   // |file_message_loop_proxy| is the MessageLoopProxy for a thread on which
   // file I/O can be done.
   JsonPrefStore(const FilePath& pref_filename,
                 base::MessageLoopProxy* file_message_loop_proxy);
   virtual ~JsonPrefStore();
 
-  // PrefStore methods:
+  // PrefStore overrides:
+  virtual ReadResult GetValue(const std::string& key, Value** result) const;
+
+  // PersistentPrefStore overrides:
+  virtual void SetValue(const std::string& key, Value* value);
+  virtual void SetValueSilently(const std::string& key, Value* value);
+  virtual void RemoveValue(const std::string& key);
   virtual bool ReadOnly() const { return read_only_; }
-
-  virtual DictionaryValue* prefs() const { return prefs_.get(); }
-
   virtual PrefReadError ReadPrefs();
-
   virtual bool WritePrefs();
-
   virtual void ScheduleWritePrefs();
 
-  // ImportantFileWriter::DataSerializer methods:
-  virtual bool SerializeData(std::string* data);
+ private:
+  // Overriden from PrefStore.

[danno, 2010/12/08 13:08:45]

Why do you make these private? By upcasting you can always call them, so I don't
see the utility.

[Mattias Nissler (ping if slow), 2010/12/09 10:20:20]

I don't care really, but I thought consumers shouldn't be interested in calling
these anyway, so they're out of the way here. Moved them up now.

+  virtual void AddObserver(PrefStore::ObserverInterface* observer);
+  virtual void RemoveObserver(PrefStore::ObserverInterface* observer);
 
- private:
+  // ImportantFileWriter::DataSerializer overrides:
+  bool SerializeData(std::string* output);
+
   FilePath path_;
 
   scoped_ptr<DictionaryValue> prefs_;
 
   bool read_only_;
 
   // Helper for safely writing pref data.
   ImportantFileWriter writer_;
+
+  ObserverList<PrefStore::ObserverInterface, true> observers_;
+
+  DISALLOW_COPY_AND_ASSIGN(JsonPrefStore);
 };
 
 #endif  // CHROME_COMMON_JSON_PREF_STORE_H_