Chromium Code Reviews

Unified Diff: chrome/browser/prefs/tracked/tracked_atomic_preference.cc

Issue 151003004: Add an automatic settings reset banner. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pre-review cleanup. Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« chrome/browser/prefs/pref_hash_store_impl.cc ('K') | « chrome/browser/prefs/pref_hash_store_impl.cc ('k') | chrome/browser/prefs/tracked/tracked_split_preference.cc » ('j') | chrome/browser/resources/options/automatic_settings_reset_banner.html » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prefs/tracked/tracked_atomic_preference.cc
diff --git a/chrome/browser/prefs/tracked/tracked_atomic_preference.cc b/chrome/browser/prefs/tracked/tracked_atomic_preference.cc
index c272fb766c2b9cd4a568933798f2f276cc1d631c..5e432f16ebe53cbdb63cf8400457eb38d2cfb0a7 100644
--- a/chrome/browser/prefs/tracked/tracked_atomic_preference.cc
+++ b/chrome/browser/prefs/tracked/tracked_atomic_preference.cc
@@ -36,8 +36,10 @@ void TrackedAtomicPreference::EnforceAndReport(
helper_.GetAction(value_state);
helper_.ReportAction(reset_action);
- if (reset_action == TrackedPreferenceHelper::DO_RESET)
+ if (reset_action == TrackedPreferenceHelper::DO_RESET) {
pref_store_contents->RemovePath(pref_path_, NULL);
+ pref_hash_store_->RecordResetEvent();
+ }
if (value_state != PrefHashStore::UNCHANGED) {
// Store the hash for the new value (whether it was reset or not).
« chrome/browser/prefs/pref_hash_store_impl.cc ('K') | « chrome/browser/prefs/pref_hash_store_impl.cc ('k') | chrome/browser/prefs/tracked/tracked_split_preference.cc » ('j') | chrome/browser/resources/options/automatic_settings_reset_banner.html » ('J')

Powered by Google App Engine