Upstreaming Select Action

content/browser/renderer_host/ime_adapter_android.h

+// 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.
+
+#ifndef CONTENT_BROWSER_RENDERER_HOST_IME_ADAPTER_ANDROID_H_
+#define CONTENT_BROWSER_RENDERER_HOST_IME_ADAPTER_ANDROID_H_
+
+#include <jni.h>
+
+namespace content {
+class RenderWidgetHostViewAndroid;
+struct NativeWebKeyboardEvent;
+}
+
+// This class has a similar role as gtk_im_context_wrapper:
+// it sits in between android IME and RenderWidgetHostViewAndroid, and manages
+// input handling for text boxes and password fields.
+//
+// This class is in charge of dispatching key events from the java side
+// and forward to renderer along with input method results via
+// corresponding host view.
+//
+// This class is used solely by RenderWidgetHostViewAndroid.
+// For more information, see http://b/issue?id=3248163

[olilan, 2012/07/31 18:24:08]

For what you've added in this CL this isn't really true. Maybe better to not
include this comment for now. Also better to not include references to http://b/

[aurimas (slooooooooow), 2012/07/31 21:12:14]

Done.

+class ImeAdapterAndroid {

[qinmin, 2012/07/31 19:22:48]

add content namespace here

[aurimas (slooooooooow), 2012/07/31 21:12:14]

Done.

+ public:
+  explicit ImeAdapterAndroid(content::RenderWidgetHostViewAndroid* rwhva);

[qinmin, 2012/07/31 19:22:48]

no content:: if ImeAdapterAndroid is in content namespace

[aurimas (slooooooooow), 2012/07/31 21:12:14]

Done.

+  ~ImeAdapterAndroid();
+
+  // Called from java -> native
+  // The java side is responsible to translate android KeyEvent various enums
+  // and values into the corresponding WebKit::WebInputEvent.
+  void Unselect(JNIEnv*, jobject);
+  void SelectAll(JNIEnv*, jobject);
+  void Cut(JNIEnv*, jobject);
+  void Copy(JNIEnv*, jobject);
+  void Paste(JNIEnv*, jobject);
+
+ private:
+  content::RenderWidgetHostViewAndroid* rwhva_;

[qinmin, 2012/07/31 19:22:48]

ditto

[aurimas (slooooooooow), 2012/07/31 21:12:14]

Done.

+  jobject java_ime_adapter_;
+};
+
+bool RegisterImeAdapter(JNIEnv* env);
+
+#endif  // CONTENT_BROWSER_RENDERER_HOST_IME_ADAPTER_ANDROID_H_