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

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

Issue 151003004: Add an automatic settings reset banner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Gab's feedback. Created 6 years, 10 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.cc
diff --git a/chrome/browser/prefs/pref_hash_filter.cc b/chrome/browser/prefs/pref_hash_filter.cc
index 1935708f9e22ca39964f2580d8dd402d1d39d9b9..99fd189fc2fe0bc2258dbe4f5415309a0a7ac420 100644
--- a/chrome/browser/prefs/pref_hash_filter.cc
+++ b/chrome/browser/prefs/pref_hash_filter.cc
@@ -4,20 +4,26 @@
#include "chrome/browser/prefs/pref_hash_filter.h"
+#include "base/callback.h"
#include "base/logging.h"
#include "base/metrics/histogram.h"
+#include "base/prefs/pref_registry_simple.h"
+#include "base/prefs/pref_service.h"
#include "base/prefs/pref_store.h"
#include "base/time/time.h"
#include "base/values.h"
+#include "chrome/browser/browser_process.h"
#include "chrome/browser/prefs/tracked/tracked_atomic_preference.h"
#include "chrome/browser/prefs/tracked/tracked_split_preference.h"
+#include "chrome/common/pref_names.h"
gab 2014/02/07 22:00:19 Remove all of the no longer necessary includes abo
robertshield 2014/02/10 02:30:53 Done.
PrefHashFilter::PrefHashFilter(
scoped_ptr<PrefHashStore> pref_hash_store,
const TrackedPreferenceMetadata tracked_preferences[],
size_t tracked_preferences_size,
size_t reporting_ids_count,
- EnforcementLevel enforcement_level) {
+ EnforcementLevel enforcement_level,
+ const base::Closure& reset_callback) : reset_callback_(reset_callback) {
DCHECK(pref_hash_store);
DCHECK_GE(reporting_ids_count, tracked_preferences_size);
@@ -82,10 +88,16 @@ void PrefHashFilter::Initialize(PrefStore* pref_store) {
void PrefHashFilter::FilterOnLoad(base::DictionaryValue* pref_store_contents) {
DCHECK(pref_store_contents);
base::TimeTicks checkpoint = base::TimeTicks::Now();
+ bool did_reset = false;
for (TrackedPreferencesMap::const_iterator it = tracked_paths_.begin();
it != tracked_paths_.end(); ++it) {
- it->second->EnforceAndReport(pref_store_contents);
+ if (it->second->EnforceAndReport(pref_store_contents))
+ did_reset = true;
}
+
+ if (did_reset)
+ reset_callback_.Run();
+
// TODO(gab): Remove this histogram by Feb 21 2014; after sufficient timing
// data has been gathered from the wild to be confident this doesn't
// significantly affect startup.

Powered by Google App Engine
This is Rietveld 408576698