Fixed the inconsistent behavior when long pressing on editable area in Chrome.

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 10d87e16fe98629050057dce80ae3b6bc182a69d..e4710a3fc1a9b2d2f081a9ceda26c1f9051b77b1 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -1967,9 +1967,16 @@ void RenderWidget::didHandleGestureEvent(
 #if defined(OS_ANDROID) || defined(USE_AURA)
   if (event_cancelled)
     return;
-  if (event.type == WebInputEvent::GestureTap ||
-      event.type == WebInputEvent::GestureLongPress) {
+  if (event.type == WebInputEvent::GestureTap) {
     UpdateTextInputState(SHOW_IME_IF_NEEDED, FROM_NON_IME);
+  } else if (event.type == WebInputEvent::GestureLongPress) {
+    blink::WebTextInputInfo new_info;
+    if (webwidget_)
+      new_info = webwidget_->textInputInfo();

[jdduke (slow), 2014/08/19 18:19:49]

Is there no other way of determining whether the input field is empty? This
query does some non-trivial work (string copying, etc...), which I suppose isn't
a huge deal as GestureLongPress is relatively infrequent, I just wonder if
there's a cheaper way of obtaining the same information.

[AKVT, 2014/08/20 15:00:43]

As I checked the code, there is no other way we can do this from existing flow,
we may have to add extra method to extract only required info. Please suggest
your opinion on this whether I can keep same or I have to write new function
from WebViewImpl.

[jdduke (slow), 2014/08/20 15:37:05]

I guess this is fine, but |didHandleGestureEvent| is a WebWidgetClient so we
should definitely still have a |webwidget_|, so you should instead do something
like:

DCHECK(webwidget_);
if (webwidget_->textInputInfo().value.isEmpty())
  UpdateTextInputState(NO_SHOW_IME, FROM_NON_IME);
else
  UpdateTextInputState(SHOW_IME_IF_NEEDED, FROM_NON_IME);

[jdduke (slow), 2014/08/20 15:37:49]

is a WebWidgetClient *method* that is.

[AKVT, 2014/08/20 18:20:17]

Done.

[AKVT, 2014/08/20 18:20:17]

Done. Thanks

+    if (!new_info.value.isEmpty())
+      UpdateTextInputState(SHOW_IME_IF_NEEDED, FROM_NON_IME);
+    else
+      UpdateTextInputState(NO_SHOW_IME, FROM_NON_IME);
   }
 #endif
 }