WebUI for Profile Settings Reset

chrome/browser/ui/browser_ui_prefs.cc

Index: chrome/browser/ui/browser_ui_prefs.cc
diff --git a/chrome/browser/ui/browser_ui_prefs.cc b/chrome/browser/ui/browser_ui_prefs.cc
index ddefee5ab2cacdbf8fdd32d38b6865848f70fc0a..425cfee6eed6f5b1c1d1a989ce74d8fffdfcbca3 100644
--- a/chrome/browser/ui/browser_ui_prefs.cc
+++ b/chrome/browser/ui/browser_ui_prefs.cc
@@ -196,6 +196,31 @@ void RegisterBrowserUserPrefs(user_prefs::PrefRegistrySyncable* registry) {
       prefs::kHideWebStoreIcon,
       false,
       user_prefs::PrefRegistrySyncable::UNSYNCABLE_PREF);
+
+  registry->RegisterBooleanPref(
+      prefs::kResetDefaultSearchEngine,
+      true,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kResetHomepage,
+      true,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kResetContentSettings,
+      true,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kResetCookiesAndSiteData,
+      true,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterBooleanPref(
+      prefs::kResetExtensions,
+      true,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
+  registry->RegisterIntegerPref(
+      prefs::kResetExtensionsHandling,
+      0,
+      user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);

[sky, 2013/05/21 15:08:29]

I haven't been involved with this feature, so forgive the question. Why do we
need prefs for these? If the user clicks the button don't we just reset the
values and not need to persist additional preferences? For example, if the user
clicks the button to reset the default home page shouldn't we just reset the
home page pref and nothing else?

[battre, 2013/05/21 16:29:31]

The reason for preferences is mostly to be consistent with the "Clear Browsing
Data" feature, where we do the same thing. If a user regularly wants to reset
only the default search engine (e.g. because some malware keeps overriding it),
it seems good to remember that preference.

 }
 
 void RegisterAppPrefs(const std::string& app_name, Profile* profile) {