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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/chromeos/preferences.cc

Issue 6905044: Refactor preference syncing. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Self review 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
« chrome/browser/autofill/autofill_manager.cc ('K') | « chrome/browser/chromeos/login/wizard_controller.cc ('k') | chrome/browser/chromeos/status/input_method_menu.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/preferences.cc
diff --git a/chrome/browser/chromeos/preferences.cc b/chrome/browser/chromeos/preferences.cc
index 6ae904f9494c363e8a5aa293baa54f263e905db9..1e295a92f04d4e5c38a0115744156d5b7f0d5e6d 100644
--- a/chrome/browser/chromeos/preferences.cc
+++ b/chrome/browser/chromeos/preferences.cc
@@ -33,100 +33,140 @@ Preferences::~Preferences() {}
// static
void Preferences::RegisterUserPrefs(PrefService* prefs) {
- prefs->RegisterBooleanPref(prefs::kTapToClickEnabled, false);
- prefs->RegisterBooleanPref(prefs::kLabsMediaplayerEnabled, false);
- prefs->RegisterBooleanPref(prefs::kLabsAdvancedFilesystemEnabled, false);
+ prefs->RegisterBooleanPref(prefs::kTapToClickEnabled,
+ false,
+ true /* sync pref */);
+ prefs->RegisterBooleanPref(prefs::kLabsMediaplayerEnabled,
+ false,
+ false /* don't sync pref */);
+ prefs->RegisterBooleanPref(prefs::kLabsAdvancedFilesystemEnabled,
+ false,
+ false /* don't sync pref */);
// Check if the accessibility pref is already registered, which can happen
// in WizardController::RegisterPrefs. We still want to try to register
// the pref here in case of Chrome/Linux with ChromeOS=1.
if (prefs->FindPreference(prefs::kAccessibilityEnabled) == NULL) {
- prefs->RegisterBooleanPref(prefs::kAccessibilityEnabled, false);
+ prefs->RegisterBooleanPref(prefs::kAccessibilityEnabled,
+ false,
+ false /* don't sync pref */);
}
- prefs->RegisterIntegerPref(prefs::kTouchpadSensitivity, 3);
- // Set the default based on the hour clock type of the current locale.
+ prefs->RegisterIntegerPref(prefs::kTouchpadSensitivity,
+ 3,
+ false /* don't sync pref */);
prefs->RegisterBooleanPref(prefs::kUse24HourClock,
- base::GetHourClockType() == base::k24HourClock);
- prefs->RegisterStringPref(prefs::kLanguageCurrentInputMethod, "");
- prefs->RegisterStringPref(prefs::kLanguagePreviousInputMethod, "");
+ base::GetHourClockType() == base::k24HourClock,
+ true /* sync pref */);
+ prefs->RegisterStringPref(prefs::kLanguageCurrentInputMethod,
+ "",
+ false /* don't sync pref */);
+ prefs->RegisterStringPref(prefs::kLanguagePreviousInputMethod,
+ "",
+ false /* don't sync pref */);
prefs->RegisterStringPref(prefs::kLanguageHotkeyNextEngineInMenu,
- language_prefs::kHotkeyNextEngineInMenu);
+ language_prefs::kHotkeyNextEngineInMenu,
+ false /* don't sync pref */);
prefs->RegisterStringPref(prefs::kLanguageHotkeyPreviousEngine,
- language_prefs::kHotkeyPreviousEngine);
+ language_prefs::kHotkeyPreviousEngine,
+ false /* don't sync pref */);
prefs->RegisterStringPref(prefs::kLanguagePreferredLanguages,
- kFallbackInputMethodLocale);
- prefs->RegisterStringPref(
- prefs::kLanguagePreloadEngines,
- input_method::GetHardwareInputMethodId());
+ kFallbackInputMethodLocale,
+ true /* sync pref */);
+ prefs->RegisterStringPref(prefs::kLanguagePreloadEngines,
+ input_method::GetHardwareInputMethodId(),
+ true /* sync pref */);
for (size_t i = 0; i < language_prefs::kNumChewingBooleanPrefs; ++i) {
prefs->RegisterBooleanPref(
language_prefs::kChewingBooleanPrefs[i].pref_name,
- language_prefs::kChewingBooleanPrefs[i].default_pref_value);
+ language_prefs::kChewingBooleanPrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
for (size_t i = 0; i < language_prefs::kNumChewingMultipleChoicePrefs; ++i) {
prefs->RegisterStringPref(
language_prefs::kChewingMultipleChoicePrefs[i].pref_name,
- language_prefs::kChewingMultipleChoicePrefs[i].default_pref_value);
+ language_prefs::kChewingMultipleChoicePrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
prefs->RegisterIntegerPref(
language_prefs::kChewingHsuSelKeyType.pref_name,
- language_prefs::kChewingHsuSelKeyType.default_pref_value);
+ language_prefs::kChewingHsuSelKeyType.default_pref_value,
+ false /* don't sync pref */);
for (size_t i = 0; i < language_prefs::kNumChewingIntegerPrefs; ++i) {
prefs->RegisterIntegerPref(
language_prefs::kChewingIntegerPrefs[i].pref_name,
- language_prefs::kChewingIntegerPrefs[i].default_pref_value);
+ language_prefs::kChewingIntegerPrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
prefs->RegisterStringPref(
prefs::kLanguageHangulKeyboard,
- language_prefs::kHangulKeyboardNameIDPairs[0].keyboard_id);
+ language_prefs::kHangulKeyboardNameIDPairs[0].keyboard_id,
+ true /* sync pref */);
prefs->RegisterStringPref(prefs::kLanguageHangulHanjaKeys,
- language_prefs::kHangulHanjaKeys);
+ language_prefs::kHangulHanjaKeys,
+ false /* don't sync pref */);
for (size_t i = 0; i < language_prefs::kNumPinyinBooleanPrefs; ++i) {
prefs->RegisterBooleanPref(
language_prefs::kPinyinBooleanPrefs[i].pref_name,
- language_prefs::kPinyinBooleanPrefs[i].default_pref_value);
+ language_prefs::kPinyinBooleanPrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
for (size_t i = 0; i < language_prefs::kNumPinyinIntegerPrefs; ++i) {
prefs->RegisterIntegerPref(
language_prefs::kPinyinIntegerPrefs[i].pref_name,
- language_prefs::kPinyinIntegerPrefs[i].default_pref_value);
+ language_prefs::kPinyinIntegerPrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
prefs->RegisterIntegerPref(
language_prefs::kPinyinDoublePinyinSchema.pref_name,
- language_prefs::kPinyinDoublePinyinSchema.default_pref_value);
+ language_prefs::kPinyinDoublePinyinSchema.default_pref_value,
+ false /* don't sync pref */);
for (size_t i = 0; i < language_prefs::kNumMozcBooleanPrefs; ++i) {
prefs->RegisterBooleanPref(
language_prefs::kMozcBooleanPrefs[i].pref_name,
- language_prefs::kMozcBooleanPrefs[i].default_pref_value);
+ language_prefs::kMozcBooleanPrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
for (size_t i = 0; i < language_prefs::kNumMozcMultipleChoicePrefs; ++i) {
prefs->RegisterStringPref(
language_prefs::kMozcMultipleChoicePrefs[i].pref_name,
- language_prefs::kMozcMultipleChoicePrefs[i].default_pref_value);
+ language_prefs::kMozcMultipleChoicePrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
for (size_t i = 0; i < language_prefs::kNumMozcIntegerPrefs; ++i) {
prefs->RegisterIntegerPref(
language_prefs::kMozcIntegerPrefs[i].pref_name,
- language_prefs::kMozcIntegerPrefs[i].default_pref_value);
+ language_prefs::kMozcIntegerPrefs[i].default_pref_value,
+ false /* don't sync pref */);
}
prefs->RegisterIntegerPref(prefs::kLanguageXkbRemapSearchKeyTo,
- input_method::kSearchKey);
+ input_method::kSearchKey,
+ true /* sync pref */);
prefs->RegisterIntegerPref(prefs::kLanguageXkbRemapControlKeyTo,
- input_method::kLeftControlKey);
+ input_method::kLeftControlKey,
+ true /* sync pref */);
prefs->RegisterIntegerPref(prefs::kLanguageXkbRemapAltKeyTo,
- input_method::kLeftAltKey);
- prefs->RegisterBooleanPref(prefs::kLanguageXkbAutoRepeatEnabled, true);
+ input_method::kLeftAltKey,
+ true /* sync pref */);
+ prefs->RegisterBooleanPref(prefs::kLanguageXkbAutoRepeatEnabled,
+ true,
+ false /* don't sync pref */);
prefs->RegisterIntegerPref(prefs::kLanguageXkbAutoRepeatDelay,
- language_prefs::kXkbAutoRepeatDelayInMs);
+ language_prefs::kXkbAutoRepeatDelayInMs,
+ false /* don't sync pref */);
prefs->RegisterIntegerPref(prefs::kLanguageXkbAutoRepeatInterval,
- language_prefs::kXkbAutoRepeatIntervalInMs);
+ language_prefs::kXkbAutoRepeatIntervalInMs,
+ false /* don't sync pref */);
// Screen lock default to off.
- prefs->RegisterBooleanPref(prefs::kEnableScreenLock, false);
+ prefs->RegisterBooleanPref(prefs::kEnableScreenLock,
+ false,
+ true /* sync pref */);
// Mobile plan notifications default to on.
- prefs->RegisterBooleanPref(prefs::kShowPlanNotifications, true);
+ prefs->RegisterBooleanPref(prefs::kShowPlanNotifications,
+ true,
+ true /* sync pref */);
}
void Preferences::Init(PrefService* prefs) {
« chrome/browser/autofill/autofill_manager.cc ('K') | « chrome/browser/chromeos/login/wizard_controller.cc ('k') | chrome/browser/chromeos/status/input_method_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698