Chrome OS: make language per user.

chrome/browser/chromeos/dom_ui/language_options_handler.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/dom_ui/language_options_handler.h"
 
 #include <map>
 #include <set>
 #include <string>
 #include <utility>
 
 #include "app/l10n_util.h"
+#include "base/basictypes.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/input_method_library.h"
 #include "chrome/browser/chromeos/input_method/input_method_util.h"
 #include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/prefs/pref_service.h"
+#include "chrome/browser/profile.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/spellcheck_common.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 
 namespace chromeos {
 
 LanguageOptionsHandler::LanguageOptionsHandler() {
 }
@@ skipping 17 matching lines @@
   localized_strings->SetString("please_add_another_input_method",
       l10n_util::GetStringUTF16(
           IDS_OPTIONS_SETTINGS_LANGUAGES_PLEASE_ADD_ANOTHER_INPUT_METHOD));
   localized_strings->SetString("please_add_another_language",
       l10n_util::GetStringUTF16(
           IDS_OPTIONS_SETTINGS_LANGUAGES_PLEASE_ADD_ANOTHER_LANGUAGE));
   localized_strings->SetString("ok_button", l10n_util::GetStringUTF16(IDS_OK));
   localized_strings->SetString("remove_button",
       l10n_util::GetStringUTF16(IDS_OPTIONS_SETTINGS_LANGUAGES_REMOVE_BUTTON));
   localized_strings->SetString("sign_out_button",
-      l10n_util::GetStringUTF16(IDS_OPTIONS_SETTINGS_LANGUAGES_SIGN_OUT_BUTTON));
+      l10n_util::GetStringUTF16(
+          IDS_OPTIONS_SETTINGS_LANGUAGES_SIGN_OUT_BUTTON));
   localized_strings->SetString("add_language_instructions",
       l10n_util::GetStringUTF16(
           IDS_OPTIONS_SETTINGS_LANGUAGES_ADD_LANGUAGE_INSTRUCTIONS));
   localized_strings->SetString("input_method_instructions",
       l10n_util::GetStringUTF16(
           IDS_OPTIONS_SETTINGS_LANGUAGES_INPUT_METHOD_INSTRUCTIONS));
   localized_strings->SetString("switch_input_methods_hint",
       l10n_util::GetStringFUTF16(
           IDS_OPTIONS_SETTINGS_LANGUAGES_SWITCH_INPUT_METHODS_HINT,
           ASCIIToUTF16("alt+shift")));
@@ skipping 209 matching lines @@
 }
 
 void LanguageOptionsHandler::UiLanguageChangeCallback(
     const ListValue* args) {
   const std::string language_code = WideToASCII(ExtractStringValue(args));
   CHECK(!language_code.empty());
   const std::string action = StringPrintf(
       "LanguageOptions_UiLanguageChange_%s", language_code.c_str());
   UserMetrics::RecordComputedAction(action);
 
-  PrefService* prefs = g_browser_process->local_state();
-  prefs->SetString(prefs::kApplicationLocale, language_code);
-  prefs->SavePersistentPrefs();
+  PrefService* prefs[] = {

[glotov, 2010/11/11 18:16:23]

Please comment why there is 2 PrefServices.

[Denis Lagno, 2010/11/12 11:51:35]

Done.

+      g_browser_process->local_state(),
+      dom_ui_->GetProfile()->GetPrefs()
+  };
+  for (size_t i = 0; i < arraysize(prefs); ++i) {
+    prefs[i]->SetString(prefs::kApplicationLocale, language_code);
+    prefs[i]->SavePersistentPrefs();
+  }
   dom_ui_->CallJavascriptFunction(
       L"options.LanguageOptions.uiLanguageSaved");
 }
 
 void LanguageOptionsHandler::SpellCheckLanguageChangeCallback(
     const ListValue* args) {
   const std::string language_code = WideToASCII(ExtractStringValue(args));
   CHECK(!language_code.empty());
   const std::string action = StringPrintf(
       "LanguageOptions_SpellCheckLanguageChange_%s", language_code.c_str());
   UserMetrics::RecordComputedAction(action);
 }
 
 void LanguageOptionsHandler::SignOutCallback(const ListValue* args) {
   UserMetrics::RecordAction(UserMetricsAction("LanguageOptions_SignOut"));
 
   Browser* browser = Browser::GetBrowserForController(
       &dom_ui_->tab_contents()->controller(), NULL);
   if (browser)
     browser->ExecuteCommand(IDC_EXIT);
 }
 
 }  // namespace chromeos