Implement the "Configure modifier keys" drop-down

chrome/browser/chromeos/options/system_page_view.cc

 // Copyright (c) 2009 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/options/system_page_view.h"
 
 #include <string>
 #include <vector>
 
 #include "app/combobox_model.h"
 #include "base/stl_util-inl.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/system_library.h"
+#include "chrome/browser/chromeos/options/language_config_util.h"
 #include "chrome/browser/chromeos/options/language_config_view.h"
 #include "chrome/browser/chromeos/options/options_window_view.h"
 #include "chrome/browser/pref_member.h"
 #include "chrome/browser/profile.h"
 #include "chrome/common/pref_names.h"
 #include "grit/generated_resources.h"
 #include "unicode/timezone.h"
 #include "views/controls/button/checkbox.h"
 #include "views/controls/button/native_button.h"
 #include "views/controls/combobox/combobox.h"
@@ skipping 317 matching lines @@
     double value =  sensitivity_.GetValue();
     sensitivity_slider_->SetValue(value);
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // LanguageSection
 
 // TextInput section for text input settings.
 class LanguageSection : public SettingsPageSection,
-                        public views::ButtonListener {
+                        public views::ButtonListener,
+                        public views::Combobox::Listener {
  public:
   explicit LanguageSection(Profile* profile);
   virtual ~LanguageSection() {}
 
  private:
   enum ButtonTag {
     kCustomizeLanguagesButton,
   };
   // Overridden from SettingsPageSection:
   virtual void InitContents(GridLayout* layout);
+  void NotifyPrefChanged(const std::wstring* pref_name);
 
   // Overridden from views::ButtonListener:
   virtual void ButtonPressed(views::Button* sender,
                              const views::Event& event);
 
+  // Overridden from views::Combobox::Listener:
+  virtual void ItemChanged(views::Combobox* sender,
+                           int prev_index,
+                           int new_index);
+
+  IntegerPrefMember xkb_pref_;
+  views::Combobox* xkb_modifier_combobox_;
+  chromeos::LanguageComboboxModel<int> xkb_modifier_combobox_model_;
+
   DISALLOW_COPY_AND_ASSIGN(LanguageSection);
 };
 
 LanguageSection::LanguageSection(Profile* profile)
     : SettingsPageSection(profile,
-                          IDS_OPTIONS_SETTINGS_SECTION_TITLE_LANGUAGE) {
+                          IDS_OPTIONS_SETTINGS_SECTION_TITLE_LANGUAGE),
+      xkb_modifier_combobox_(NULL),
+      xkb_modifier_combobox_model_(&kXkbModifierMultipleChoicePrefs) {
+  xkb_pref_.Init(prefs::kLanguageXkbModifierRemap, profile->GetPrefs(), this);
 }
 
 void LanguageSection::InitContents(GridLayout* layout) {
-  // Add the customize button.
-  layout->StartRow(0, single_column_view_set_id());
+  // Add the customize button and XKB combobox.
+  layout->StartRow(0, double_column_view_set_id());
   views::NativeButton* customize_languages_button = new views::NativeButton(
       this,
       l10n_util::GetString(IDS_OPTIONS_SETTINGS_LANGUAGES_CUSTOMIZE));
   customize_languages_button->set_tag(kCustomizeLanguagesButton);
   layout->AddView(customize_languages_button, 1, 1,
                   GridLayout::LEADING, GridLayout::CENTER);
+
+  xkb_modifier_combobox_ = new views::Combobox(&xkb_modifier_combobox_model_);
+  xkb_modifier_combobox_->set_listener(this);
+  // Initialize the combobox to what's saved in user preferences. Otherwise,
+  // ItemChanged() will be called with |new_index| == 0.
+  NotifyPrefChanged(NULL);
+
+  layout->AddView(xkb_modifier_combobox_);
   layout->AddPaddingRow(0, kUnrelatedControlVerticalSpacing);
 }
 
 void LanguageSection::ButtonPressed(
     views::Button* sender, const views::Event& event) {
   if (sender->tag() == kCustomizeLanguagesButton) {
     LanguageConfigView::Show(profile(), GetOptionsViewParent());
   }
 }
 
+void LanguageSection::ItemChanged(views::Combobox* sender,
+                                  int prev_index,
+                                  int new_index) {
+  LOG(INFO) << "Changing XKB modofier pref to " << new_index;
+  xkb_pref_.SetValue(new_index);
+}
+
+void LanguageSection::NotifyPrefChanged(const std::wstring* pref_name) {
+  if (!pref_name || *pref_name == prefs::kLanguageXkbModifierRemap) {
+    const int id = xkb_pref_.GetValue();
+    if (id >= 0) {
+      xkb_modifier_combobox_->SetSelectedItem(id);
+    }
+  }
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // AccessibilitySection
 
 // Checkbox for specifying if accessibility should be enabled for this profile
 class AccessibilitySection : public SettingsPageSection,
                              public views::ButtonListener {
  public:
   explicit AccessibilitySection(Profile* profile);
   virtual ~AccessibilitySection() {}
 
@@ skipping 79 matching lines @@
   layout->AddPaddingRow(0, kRelatedControlVerticalSpacing);
   layout->StartRow(0, single_column_view_set_id);
   layout->AddView(new LanguageSection(profile()));
   layout->AddPaddingRow(0, kRelatedControlVerticalSpacing);
   layout->StartRow(0, single_column_view_set_id);
   layout->AddView(new AccessibilitySection(profile()));
   layout->AddPaddingRow(0, kRelatedControlVerticalSpacing);
 }
 
 }  // namespace chromeos