[Android] Should not update IME status when gesture needs to be disambiguated

content/renderer/render_widget.cc

Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index ff0cc11f13f6f501e3e816fee1b725fa9cf6dd18..2be8d45f31982a2381132c0fc453c9c8131074c1 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -1958,6 +1958,19 @@ void RenderWidget::resetInputMethod() {
   UpdateCompositionInfo(range, std::vector<gfx::Rect>());
 }
 
+void RenderWidget::didHandleGestureEvent(
+    const WebGestureEvent& event,
+    EventStatus event_status) {
+#if defined(OS_ANDROID)
+  if (event_status == EventStatusCancelled)

[darin (slow to review), 2013/01/15 23:06:03]

why did you plumb an EventStatus enum if you only needed to know if the event
was cancelled?

+    return;
+  if (event.type == WebInputEvent::GestureTap ||
+      event.type == WebInputEvent::GestureLongPress) {
+    UpdateTextInputState(SHOW_IME_IF_NEEDED);
+  }
+#endif
+}
+
 void RenderWidget::SchedulePluginMove(
     const webkit::npapi::WebPluginGeometry& move) {
   size_t i = 0;