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

Unified Diff: chrome/browser/download/download_prefs.cc

Issue 6905044: Refactor preference syncing. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + comments Created 9 years, 8 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/download/download_prefs.cc
diff --git a/chrome/browser/download/download_prefs.cc b/chrome/browser/download/download_prefs.cc
index 6d1cd67538a623244d7a95fa65323b04d2025f57..c13cadf2baeb3b87a936d01e7ee8152ee241659c 100644
--- a/chrome/browser/download/download_prefs.cc
+++ b/chrome/browser/download/download_prefs.cc
@@ -46,17 +46,25 @@ DownloadPrefs::~DownloadPrefs() {
// static
void DownloadPrefs::RegisterUserPrefs(PrefService* prefs) {
- prefs->RegisterBooleanPref(prefs::kPromptForDownload, false);
- prefs->RegisterStringPref(prefs::kDownloadExtensionsToOpen, "");
- prefs->RegisterBooleanPref(prefs::kDownloadDirUpgraded, false);
+ prefs->RegisterBooleanPref(prefs::kPromptForDownload,
+ false,
+ true /* sync pref */);
+ prefs->RegisterStringPref(prefs::kDownloadExtensionsToOpen,
+ "",
+ false /* don't sync pref */);
+ prefs->RegisterBooleanPref(prefs::kDownloadDirUpgraded,
+ false,
+ false /* don't sync pref */);
prefs->RegisterIntegerPref(prefs::kSaveFileType,
- SavePackage::SAVE_AS_COMPLETE_HTML);
+ SavePackage::SAVE_AS_COMPLETE_HTML,
+ false /* don't sync pref */);
// The default download path is userprofile\download.
const FilePath& default_download_path =
download_util::GetDefaultDownloadDirectory();
prefs->RegisterFilePathPref(prefs::kDownloadDefaultDirectory,
- default_download_path);
+ default_download_path,
+ false /* don't sync pref */);
#if defined(OS_CHROMEOS)
// Ensure that the download directory specified in the preferences exists.

Powered by Google App Engine
This is Rietveld 408576698