Adding a configuration dialog for Pinyin input method.

chrome/browser/chromeos/options/language_pinyin_config_view.h

+// 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.
+
+#ifndef CHROME_BROWSER_CHROMEOS_OPTIONS_LANGUAGE_PINYIN_CONFIG_VIEW_H_
+#define CHROME_BROWSER_CHROMEOS_OPTIONS_LANGUAGE_PINYIN_CONFIG_VIEW_H_
+
+#include <string>
+
+#include "chrome/browser/chromeos/cros/language_library.h"
+#include "chrome/browser/chromeos/language_preferences.h"
+#include "chrome/browser/pref_member.h"
+#include "chrome/browser/views/options/options_page_view.h"
+#include "views/controls/button/checkbox.h"
+#include "views/controls/label.h"
+#include "views/window/dialog_delegate.h"
+
+namespace chromeos {
+
+// A dialog box for showing Traditional Chinese (Pinyin) input method
+// preferences.
+class LanguagePinyinConfigView : public views::ButtonListener,
+                                 public views::DialogDelegate,
+                                 public OptionsPageView {
+ public:
+  explicit LanguagePinyinConfigView(Profile* profile);
+  virtual ~LanguagePinyinConfigView();
+
+  // views::ButtonListener overrides.
+  virtual void ButtonPressed(views::Button* sender, const views::Event& event);
+
+  // views::DialogDelegate overrides.
+  virtual bool IsModal() const { return true; }
+  virtual views::View* GetContentsView() { return this; }
+  virtual std::wstring GetWindowTitle() const;
+
+  // views::View overrides.
+  virtual void Layout();
+  virtual gfx::Size GetPreferredSize();
+
+  // OptionsPageView overrides.
+  virtual void InitControlLayout();
+
+  // NotificationObserver overrides.
+  virtual void Observe(NotificationType type,
+                       const NotificationSource& source,
+                       const NotificationDetails& details);
+
+ private:
+  // Updates the pinyin checkboxes.
+  void NotifyPrefChanged();
+
+  BooleanPrefMember pinyin_boolean_prefs_[kNumPinyinBooleanPrefs];
+  // TODO(yusukes): Support integer prefs if needed.

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

I guess this belongs to |pinyin_boolean_prefs_|, if so could move it above?
(otherwise it looks it's talking about |contents_|)

+  views::View* contents_;
+
+  // A checkboxes for Pinyin.

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

nit: The checkboxes?

+  views::Checkbox* pinyin_boolean_checkboxes_[kNumPinyinBooleanPrefs];
+
+  DISALLOW_COPY_AND_ASSIGN(LanguagePinyinConfigView);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_OPTIONS_LANGUAGE_PINYIN_CONFIG_VIEW_H_