Adding a configuration dialog for Pinyin input method.

chrome/browser/chromeos/preferences.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.

[tfarina (gmail-do not use), 2010/04/27 18:14:21]

nit: 2010

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/preferences.h"
 
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/language_library.h"
 #include "chrome/browser/chromeos/cros/synaptics_library.h"
@@ skipping 13 matching lines @@
   prefs->RegisterIntegerPref(prefs::kTouchpadSpeedFactor, 5);
   prefs->RegisterIntegerPref(prefs::kTouchpadSensitivity, 5);
   prefs->RegisterBooleanPref(prefs::kLanguageUseGlobalEngine, true);
   prefs->RegisterStringPref(prefs::kLanguageHotkeyNextEngine,
                             kHotkeyNextEngine);
   prefs->RegisterStringPref(prefs::kLanguageHotkeyTrigger, kHotkeyTrigger);
   prefs->RegisterStringPref(prefs::kLanguagePreloadEngines,
                             UTF8ToWide(kFallbackInputMethodId));  // EN layout
   prefs->RegisterStringPref(prefs::kLanguageHangulKeyboard,
                             kHangulKeyboardNameIDPairs[0].keyboard_id);
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPinyinBooleanPrefs); ++i) {
+  for (size_t i = 0; i < kNumPinyinBooleanPrefs; ++i) {
     prefs->RegisterBooleanPref(kPinyinBooleanPrefs[i].pref_name,
                                kPinyinBooleanPrefs[i].default_value);
   }
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPinyinIntegerPrefs); ++i) {
+  for (size_t i = 0; i < kNumPinyinIntegerPrefs; ++i) {
     prefs->RegisterIntegerPref(kPinyinIntegerPrefs[i].pref_name,
                                kPinyinIntegerPrefs[i].default_value);
   }
 }
 
 void Preferences::Init(PrefService* prefs) {
   timezone_.Init(prefs::kTimeZone, prefs, this);
   tap_to_click_enabled_.Init(prefs::kTapToClickEnabled, prefs, this);
   vert_edge_scroll_enabled_.Init(prefs::kVertEdgeScrollEnabled, prefs, this);
   speed_factor_.Init(prefs::kTouchpadSpeedFactor, prefs, this);
   sensitivity_.Init(prefs::kTouchpadSensitivity, prefs, this);
   language_use_global_engine_.Init(
       prefs::kLanguageUseGlobalEngine, prefs, this);
   language_hotkey_next_engine_.Init(
       prefs::kLanguageHotkeyNextEngine, prefs, this);
   language_hotkey_trigger_.Init(prefs::kLanguageHotkeyTrigger, prefs, this);
   language_preload_engines_.Init(prefs::kLanguagePreloadEngines, prefs, this);
   language_hangul_keyboard_.Init(prefs::kLanguageHangulKeyboard, prefs, this);
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPinyinBooleanPrefs); ++i) {
+  for (size_t i = 0; i < kNumPinyinBooleanPrefs; ++i) {
     language_pinyin_boolean_prefs_[i].Init(
         kPinyinBooleanPrefs[i].pref_name, prefs, this);
   }
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPinyinIntegerPrefs); ++i) {
+  for (size_t i = 0; i < kNumPinyinIntegerPrefs; ++i) {
     language_pinyin_int_prefs_[i].Init(
         kPinyinIntegerPrefs[i].pref_name, prefs, this);
   }
 
   // Initialize touchpad settings to what's saved in user preferences.
   NotifyPrefChanged(NULL);
 }
 
 void Preferences::Observe(NotificationType type,
                           const NotificationSource& source,
@@ skipping 32 matching lines @@
     SetLanguageConfigStringListAsCSV(kHotKeySectionName,
                                      kTriggerConfigName,
                                      language_hotkey_trigger_.GetValue());
   if (!pref_name || *pref_name == prefs::kLanguagePreloadEngines)
     SetLanguageConfigStringListAsCSV(kGeneralSectionName,
                                      kPreloadEnginesConfigName,
                                      language_preload_engines_.GetValue());
   if (!pref_name || *pref_name == prefs::kLanguageHangulKeyboard)
     SetLanguageConfigString(kHangulSectionName, kHangulKeyboardConfigName,
                             language_hangul_keyboard_.GetValue());
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPinyinBooleanPrefs); ++i) {
+  for (size_t i = 0; i < kNumPinyinBooleanPrefs; ++i) {
     if (!pref_name || *pref_name == kPinyinBooleanPrefs[i].pref_name) {
       SetLanguageConfigBoolean(kPinyinSectionName,
                                kPinyinBooleanPrefs[i].ibus_config_name,
                                language_pinyin_boolean_prefs_[i].GetValue());
     }
   }
-  for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kPinyinIntegerPrefs); ++i) {
+  for (size_t i = 0; i < kNumPinyinIntegerPrefs; ++i) {
     if (!pref_name || *pref_name == kPinyinIntegerPrefs[i].pref_name) {
       SetLanguageConfigInteger(kPinyinSectionName,
                                kPinyinIntegerPrefs[i].ibus_config_name,
                                language_pinyin_int_prefs_[i].GetValue());
     }
   }
 }
 
 void Preferences::SetTimeZone(const std::wstring& id) {
   icu::TimeZone* timezone = icu::TimeZone::createTimeZone(
@@ skipping 48 matching lines @@
   std::vector<std::wstring> split_values;
   if (!value.empty()) {
     SplitString(value, L',', &split_values);
   }
   // We should call the cros API even when |value| is empty, to disable default
   // config.
   SetLanguageConfigStringList(section, name, split_values);
 }
 
 }  // namespace chromeos