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

Unified Diff: chrome/browser/prefs/pref_hash_filter.h

Issue 266553002: Add TrackedPreferenceValidationDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix expectations for android and cros Created 6 years, 7 months 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/prefs/pref_hash_filter.h
diff --git a/chrome/browser/prefs/pref_hash_filter.h b/chrome/browser/prefs/pref_hash_filter.h
index 7cb08313d0615faf26bb9bccdc124412af3fdcb0..1ea0da769fb504b62601e57e248519a4ef65e8eb 100644
--- a/chrome/browser/prefs/pref_hash_filter.h
+++ b/chrome/browser/prefs/pref_hash_filter.h
@@ -21,6 +21,7 @@
class PersistentPrefStore;
class PrefService;
class PrefStore;
+class TrackedPreferenceValidationObserver;
namespace base {
class DictionaryValue;
@@ -59,12 +60,14 @@ class PrefHashFilter : public PrefFilter {
};
// Constructs a PrefHashFilter tracking the specified |tracked_preferences|
- // using |pref_hash_store| to check/store hashes.
+ // using |pref_hash_store| to check/store hashes. |observer| is notified of
+ // the status of each preference as it is checked.
// |reporting_ids_count| is the count of all possible IDs (possibly greater
// than |tracked_preferences.size()|).
PrefHashFilter(
scoped_ptr<PrefHashStore> pref_hash_store,
const std::vector<TrackedPreferenceMetadata>& tracked_preferences,
+ scoped_ptr<TrackedPreferenceValidationObserver> observer,
size_t reporting_ids_count);
virtual ~PrefHashFilter();
@@ -111,6 +114,8 @@ class PrefHashFilter : public PrefFilter {
scoped_ptr<PrefHashStore> pref_hash_store_;
+ scoped_ptr<TrackedPreferenceValidationObserver> observer_;
+
TrackedPreferencesMap tracked_paths_;
// The set of all paths whose value has changed since the last call to

Powered by Google App Engine
This is Rietveld 408576698