Expand VPD initial_locale to a list of locales. Use the expanded VPD initial_locale on the OOBE.

chrome/browser/ui/webui/options/chromeos/cros_language_options_handler.cc

 // Copyright (c) 2012 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/ui/webui/options/chromeos/cros_language_options_handler.h"
 
+#include <algorithm>
+#include <iterator>
 #include <map>
 #include <set>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/i18n/rtl.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/chromeos/customization_document.h"
 #include "chrome/browser/chromeos/input_method/input_method_util.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
@@ skipping 148 matching lines @@
 
     input_method_list->Append(dictionary);
   }
 
   return input_method_list;
 }
 
 // static
 base::ListValue* CrosLanguageOptionsHandler::GetLanguageListInternal(
     const input_method::InputMethodDescriptors& descriptors,
-    const std::vector<std::string>& base_language_codes) {
+    const std::vector<std::string>& base_language_codes,
+    const bool insert_divider) {
   const std::string app_locale = g_browser_process->GetApplicationLocale();
 
   std::set<std::string> language_codes;
   // Collect the language codes from the supported input methods.
   for (size_t i = 0; i < descriptors.size(); ++i) {
     const input_method::InputMethodDescriptor& descriptor = descriptors[i];
     const std::vector<std::string>& languages =
         descriptor.language_codes();
     for (size_t i = 0; i < languages.size(); ++i)
       language_codes.insert(languages[i]);
   }
 
+  const StartupCustomizationDocument* startup_manifest =
+      StartupCustomizationDocument::GetInstance();
+
+  const std::vector<std::string>& configured_locales =
+      startup_manifest->configured_locales();
+
+  // Languages sort order.
+  std::map<std::string, int /* index */> language_index;
+  for (size_t i = 0; i < configured_locales.size(); ++i) {
+    language_index[configured_locales[i]] = i;
+  }
+
   // Map of display name -> {language code, native_display_name}.
   // In theory, we should be able to create a map that is sorted by
   // display names using ICU comparator, but doing it is hard, thus we'll
   // use an auxiliary vector to achieve the same result.
   typedef std::pair<std::string, base::string16> LanguagePair;
   typedef std::map<base::string16, LanguagePair> LanguageMap;
   LanguageMap language_map;
-  // The auxiliary vector mentioned above.
+  // The auxiliary vector mentioned above. (except vendor locales)

[Nikita (slow), 2014/01/30 13:23:28]

nit: Insert empty line before comment (and below).

[Alexander Alekseev, 2014/01/31 12:46:54]

Done.

   std::vector<base::string16> display_names;
+  // Separate vector of vendor locales.
+  std::vector<base::string16> configured_locales_display_names(
+      configured_locales.size());
 
+  size_t configured_locales_count = 0;
   // Build the list of display names, and build the language map.
   for (std::set<std::string>::const_iterator iter = language_codes.begin();
        iter != language_codes.end(); ++iter) {
      // Exclude the language which is not in |base_langauge_codes| even it has
      // input methods.
     if (std::find(base_language_codes.begin(),
                   base_language_codes.end(),
                   *iter) == base_language_codes.end()) {
       continue;
     }
 
     const base::string16 display_name =
         l10n_util::GetDisplayNameForLocale(*iter, app_locale, true);
     const base::string16 native_display_name =
         l10n_util::GetDisplayNameForLocale(*iter, *iter, true);
 
-    display_names.push_back(display_name);
     language_map[display_name] =
         std::make_pair(*iter, native_display_name);
+
+    const std::map<std::string, int>::const_iterator index_pos =
+        language_index.find(*iter);
+    if (index_pos != language_index.end()) {
+      configured_locales_display_names[index_pos->second] = display_name;
+      ++configured_locales_count;
+    } else {
+      display_names.push_back(display_name);
+    }
   }
-  DCHECK_EQ(display_names.size(), language_map.size());
+  DCHECK_EQ(display_names.size() + configured_locales_count,
+            language_map.size());
 
   // Build the list of display names, and build the language map.
   for (size_t i = 0; i < base_language_codes.size(); ++i) {
     // Skip this language if it was already added.
     if (language_codes.find(base_language_codes[i]) != language_codes.end())
       continue;
 
     // TODO(zork): Remove this blacklist when fonts are added to Chrome OS.
     // see: crbug.com/240586
     if (IsBlacklisted(base_language_codes[i]))
       continue;
 
     base::string16 display_name =
         l10n_util::GetDisplayNameForLocale(
             base_language_codes[i], app_locale, false);
     base::string16 native_display_name =
         l10n_util::GetDisplayNameForLocale(
             base_language_codes[i], base_language_codes[i], false);
     display_names.push_back(display_name);
     language_map[display_name] =
         std::make_pair(base_language_codes[i], native_display_name);
+
+    const std::map<std::string, int>::const_iterator index_pos =
+        language_index.find(base_language_codes[i]);
+    if (index_pos != language_index.end()) {
+      configured_locales_display_names[index_pos->second] = display_name;
+      ++configured_locales_count;
+    } else {
+      display_names.push_back(display_name);
+    }
   }
 
   // Sort display names using locale specific sorter.
   l10n_util::SortStrings16(app_locale, &display_names);
+  // Concatenate configured_locales_display_names and display_names.
+  // Insert empty string as a divider.
+  std::vector<base::string16> out_display_names;
+  for (size_t i = 0; i < configured_locales_display_names.size(); ++i) {
+    if (configured_locales_display_names[i].size() == 0)
+      continue;
+    out_display_names.push_back(configured_locales_display_names[i]);
+  }
+  if (insert_divider)
+    out_display_names.push_back(base::string16());
+  std::copy(display_names.begin(),
+            display_names.end(),
+            std::back_inserter(out_display_names));
 
   // Build the language list from the language map.
   base::ListValue* language_list = new base::ListValue();
-  for (size_t i = 0; i < display_names.size(); ++i) {
+  for (size_t i = 0; i < out_display_names.size(); ++i) {
     // Sets the directionality of the display language name.
-    base::string16 display_name(display_names[i]);
+    base::string16 display_name(out_display_names[i]);
+    if (insert_divider && display_name.empty()) {
+      // Insert divider.
+      base::DictionaryValue* dictionary = new base::DictionaryValue();
+      dictionary->SetString("code", "");

[Nikita (slow), 2014/01/30 13:23:28]

nit: As discussed, let's use a specific string constant that will mean a divider
element. This way if someone inserts invalid input_method manager code (empty
string) that will not become a divider.

[Alexander Alekseev, 2014/01/31 12:46:54]

Done.

+      language_list->Append(dictionary);
+      continue;
+    }
     bool markup_removal =
         base::i18n::UnadjustStringForLocaleDirection(&display_name);
     DCHECK(markup_removal);
     bool has_rtl_chars = base::i18n::StringContainsStrongRTLChars(display_name);
     std::string directionality = has_rtl_chars ? "rtl" : "ltr";
 
-    const LanguagePair& pair = language_map[display_names[i]];
+    const LanguagePair& pair = language_map[out_display_names[i]];
     base::DictionaryValue* dictionary = new base::DictionaryValue();
     dictionary->SetString("code", pair.first);
-    dictionary->SetString("displayName", display_names[i]);
+    dictionary->SetString("displayName", out_display_names[i]);
     dictionary->SetString("textDirection", directionality);
     dictionary->SetString("nativeDisplayName", pair.second);
     language_list->Append(dictionary);
   }
 
   return language_list;
 }
 
 // static
 base::ListValue* CrosLanguageOptionsHandler::GetAcceptLanguageList(
     const input_method::InputMethodDescriptors& descriptors) {
   // Collect the language codes from the supported accept-languages.
   const std::string app_locale = g_browser_process->GetApplicationLocale();
   std::vector<std::string> accept_language_codes;
   l10n_util::GetAcceptLanguagesForLocale(app_locale, &accept_language_codes);
-  return GetLanguageListInternal(descriptors, accept_language_codes);
+  return GetLanguageListInternal(descriptors, accept_language_codes, false);
 }
 
 // static
 base::ListValue* CrosLanguageOptionsHandler::GetUILanguageList(
     const input_method::InputMethodDescriptors& descriptors) {
   // Collect the language codes from the available locales.
-  return GetLanguageListInternal(descriptors, l10n_util::GetAvailableLocales());
+  return GetLanguageListInternal(
+      descriptors, l10n_util::GetAvailableLocales(), true);
 }
 
 base::ListValue*
     CrosLanguageOptionsHandler::ConvertInputMethodDescriptosToIMEList(
         const input_method::InputMethodDescriptors& descriptors) {
   scoped_ptr<base::ListValue> ime_ids_list(new base::ListValue());
   for (size_t i = 0; i < descriptors.size(); ++i) {
     const input_method::InputMethodDescriptor& descriptor = descriptors[i];
     scoped_ptr<base::DictionaryValue> dictionary(new base::DictionaryValue());
     dictionary->SetString("id", descriptor.id());
@@ skipping 111 matching lines @@
       ConvertInputMethodDescriptosToIMEList(
           component_extension_manager->GetAllIMEAsInputMethodDescriptor()));
   web_ui()->CallJavascriptFunction(
       "options.LanguageOptions.onComponentManagerInitialized",
       *ime_list);
   composition_extension_appended_ = true;
 }
 
 }  // namespace options
 }  // namespace chromeos