Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Side by Side Diff: base/prefs/value_map_pref_store.h

Issue 11358123: Rewrite DefaultPrefStore for performance (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Code review changes Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BASE_PREFS_VALUE_MAP_PREF_STORE_H_ 5 #ifndef BASE_PREFS_VALUE_MAP_PREF_STORE_H_
6 #define BASE_PREFS_VALUE_MAP_PREF_STORE_H_ 6 #define BASE_PREFS_VALUE_MAP_PREF_STORE_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
11 #include "base/basictypes.h" 11 #include "base/basictypes.h"
12 #include "base/observer_list.h" 12 #include "base/observer_list.h"
13 #include "base/prefs/base_prefs_export.h" 13 #include "base/prefs/base_prefs_export.h"
14 #include "base/prefs/pref_store.h" 14 #include "base/prefs/pref_store.h"
15 #include "base/prefs/pref_value_map.h" 15 #include "base/prefs/pref_value_map.h"
16 16
17 // A basic PrefStore implementation that uses a simple name-value map for 17 // A basic PrefStore implementation that uses a simple name-value map for
18 // storing the preference values. 18 // storing the preference values.
19 class BASE_PREFS_EXPORT ValueMapPrefStore : public PrefStore { 19 class BASE_PREFS_EXPORT ValueMapPrefStore : public PrefStore {
20 public: 20 public:
21 typedef std::map<std::string, base::Value*>::iterator iterator;
22 typedef std::map<std::string, base::Value*>::const_iterator const_iterator;
23
24 ValueMapPrefStore(); 21 ValueMapPrefStore();
25 22
26 // PrefStore overrides: 23 // PrefStore overrides:
27 virtual ReadResult GetValue(const std::string& key, 24 virtual ReadResult GetValue(const std::string& key,
28 const base::Value** value) const OVERRIDE; 25 const base::Value** value) const OVERRIDE;
29 virtual void AddObserver(PrefStore::Observer* observer) OVERRIDE; 26 virtual void AddObserver(PrefStore::Observer* observer) OVERRIDE;
30 virtual void RemoveObserver(PrefStore::Observer* observer) OVERRIDE; 27 virtual void RemoveObserver(PrefStore::Observer* observer) OVERRIDE;
31 virtual size_t NumberOfObservers() const OVERRIDE; 28 virtual size_t NumberOfObservers() const OVERRIDE;
32 29
33 iterator begin();
34 iterator end();
35 const_iterator begin() const;
36 const_iterator end() const;
37
38 protected: 30 protected:
39 virtual ~ValueMapPrefStore(); 31 virtual ~ValueMapPrefStore();
40 32
41 // Store a |value| for |key| in the store. Also generates an notification if 33 // Store a |value| for |key| in the store. Also generates an notification if
42 // the value changed. Assumes ownership of |value|, which must be non-NULL. 34 // the value changed. Assumes ownership of |value|, which must be non-NULL.
43 void SetValue(const std::string& key, base::Value* value); 35 void SetValue(const std::string& key, base::Value* value);
44 36
45 // Remove the value for |key| from the store. Sends a notification if there 37 // Remove the value for |key| from the store. Sends a notification if there
46 // was a value to be removed. 38 // was a value to be removed.
47 void RemoveValue(const std::string& key); 39 void RemoveValue(const std::string& key);
48 40
49 // Notify observers about the initialization completed event. 41 // Notify observers about the initialization completed event.
50 void NotifyInitializationCompleted(); 42 void NotifyInitializationCompleted();
51 43
52 private: 44 private:
53 PrefValueMap prefs_; 45 PrefValueMap prefs_;
54 46
55 ObserverList<PrefStore::Observer, true> observers_; 47 ObserverList<PrefStore::Observer, true> observers_;
56 48
57 DISALLOW_COPY_AND_ASSIGN(ValueMapPrefStore); 49 DISALLOW_COPY_AND_ASSIGN(ValueMapPrefStore);
58 }; 50 };
59 51
60 #endif // BASE_PREFS_VALUE_MAP_PREF_STORE_H_ 52 #endif // BASE_PREFS_VALUE_MAP_PREF_STORE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698