MacViews: Implement a ui::InputMethod for Mac

ui/base/ime/input_method_mac.mm

Index: ui/base/ime/input_method_mac.mm
diff --git a/ui/base/ime/input_method_mac.mm b/ui/base/ime/input_method_mac.mm
new file mode 100644
index 0000000000000000000000000000000000000000..c4b55c984d93082e905544ba56eac4e885979be3
--- /dev/null
+++ b/ui/base/ime/input_method_mac.mm
@@ -0,0 +1,49 @@
+// Copyright 2014 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 "ui/base/ime/input_method_mac.h"
+
+namespace ui {
+
+InputMethodMac::InputMethodMac(internal::InputMethodDelegate* delegate) {
+  SetDelegate(delegate);
+}
+
+InputMethodMac::~InputMethodMac() {
+}
+
+bool InputMethodMac::OnUntranslatedIMEMessage(const base::NativeEvent& event,
+                                              NativeEventResult* result) {
+  return false;
+}
+
+bool InputMethodMac::DispatchKeyEvent(const ui::KeyEvent& event) {
+  // IME processing on the Mac does not go through this path.
+  NOTREACHED();
+  return false;
+}
+
+void InputMethodMac::OnCaretBoundsChanged(const TextInputClient* client) {
+}
+
+void InputMethodMac::CancelComposition(const TextInputClient* client) {
+}
+
+void InputMethodMac::OnInputLocaleChanged() {
+}
+
+std::string InputMethodMac::GetInputLocale() {
+  return "";
+}
+
+bool InputMethodMac::IsActive() {
+  return true;

[Yuki, 2014/06/03 07:50:15]

I'm not sure if you really want to return true or not.
You may want to return false because your InputMethodMac doesn't work, or you
may want to return true because IME (NSTextInputClient or something) works on
Mac, too.

[Andre, 2014/06/03 17:55:03]

I agree, it's not clear.
This will not be called on the Mac because the only call site is in
RenderWidgetHostViewAura.
I'm thinking true is better for now.

+}
+
+bool InputMethodMac::IsCandidatePopupOpen() const {
+  // There seems to be no way to tell if a candidate window is open.
+  return false;
+}
+
+}  // namespace ui