diff options
author | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-26 18:26:12 +0000 |
---|---|---|
committer | mattm@chromium.org <mattm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-26 18:26:12 +0000 |
commit | a33f62b19e7be82d6784df99ce7cfc5321910a07 (patch) | |
tree | 439f5e9376d1cde395f52994399822e982a734b9 /chrome/browser/views/options | |
parent | daccbce9086a33e84f1eaec42b9781e7f137fa8a (diff) | |
download | chromium_src-a33f62b19e7be82d6784df99ce7cfc5321910a07.zip chromium_src-a33f62b19e7be82d6784df99ce7cfc5321910a07.tar.gz chromium_src-a33f62b19e7be82d6784df99ce7cfc5321910a07.tar.bz2 |
move the ResetToDefault function for prefs to chrome/browser/options_util
BUG=11507
Review URL: http://codereview.chromium.org/147190
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19390 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/options')
-rw-r--r-- | chrome/browser/views/options/advanced_page_view.cc | 56 |
1 files changed, 2 insertions, 54 deletions
diff --git a/chrome/browser/views/options/advanced_page_view.cc b/chrome/browser/views/options/advanced_page_view.cc index 4d4ccad..f5136342 100644 --- a/chrome/browser/views/options/advanced_page_view.cc +++ b/chrome/browser/views/options/advanced_page_view.cc @@ -8,11 +8,9 @@ #include "app/message_box_flags.h" #include "base/string_util.h" #include "base/values.h" -#include "chrome/browser/browser_process.h" +#include "chrome/browser/options_util.h" #include "chrome/browser/views/options/advanced_contents_view.h" #include "chrome/common/chrome_constants.h" -#include "chrome/common/pref_names.h" -#include "chrome/common/pref_service.h" #include "grit/chromium_strings.h" #include "grit/generated_resources.h" #include "grit/locale_settings.h" @@ -98,57 +96,7 @@ AdvancedPageView::~AdvancedPageView() { } void AdvancedPageView::ResetToDefaults() { - // TODO(tc): It would be nice if we could generate this list automatically so - // changes to any of the options pages doesn't require updating this list - // manually. - PrefService* prefs = profile()->GetPrefs(); - const wchar_t* kUserPrefs[] = { - prefs::kAcceptLanguages, - prefs::kAlternateErrorPagesEnabled, - prefs::kCookieBehavior, - prefs::kDefaultCharset, - prefs::kDnsPrefetchingEnabled, - prefs::kDownloadDefaultDirectory, - prefs::kDownloadExtensionsToOpen, - prefs::kFormAutofillEnabled, - prefs::kHomePage, - prefs::kHomePageIsNewTabPage, - prefs::kMixedContentFiltering, - prefs::kPromptForDownload, - prefs::kPasswordManagerEnabled, - prefs::kRestoreOnStartup, - prefs::kSafeBrowsingEnabled, - prefs::kSearchSuggestEnabled, - prefs::kShowHomeButton, - prefs::kSpellCheckDictionary, - prefs::kURLsToRestoreOnStartup, - prefs::kWebKitDefaultFixedFontSize, - prefs::kWebKitDefaultFontSize, - prefs::kWebKitFixedFontFamily, - prefs::kWebKitJavaEnabled, - prefs::kWebKitJavascriptEnabled, - prefs::kWebKitLoadsImagesAutomatically, - prefs::kWebKitPluginsEnabled, - prefs::kWebKitSansSerifFontFamily, - prefs::kWebKitSerifFontFamily, - }; - for (int i = 0; i < arraysize(kUserPrefs); ++i) - prefs->ClearPref(kUserPrefs[i]); - - PrefService* local_state = g_browser_process->local_state(); - // Note that we don't reset the kMetricsReportingEnabled preference here - // because the reset will reset it to the default setting specified in Chrome - // source, not the default setting selected by the user on the web page where - // they downloaded Chrome. This means that if the user ever resets their - // settings they'll either inadvertedly enable this logging or disable it. - // One is undesirable for them, one is undesirable for us. For now, we just - // don't reset it. - const wchar_t* kLocalStatePrefs[] = { - prefs::kApplicationLocale, - prefs::kOptionsWindowLastTabIndex, - }; - for (int i = 0; i < arraysize(kLocalStatePrefs); ++i) - local_state->ClearPref(kLocalStatePrefs[i]); + OptionsUtil::ResetToDefaults(profile()); } /////////////////////////////////////////////////////////////////////////////// |