Set the Insertion handle correctly while showing the paste popup menu

content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java

Index: content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java
diff --git a/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java b/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java
index e63937fd94cd00c64407fe51eb113c41a9ee5cfe..4733bdd1db9553038cd63aa00fb9a4af720492c2 100644
--- a/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java
+++ b/content/public/android/javatests/src/org/chromium/content/browser/input/ImeTest.java
@@ -540,6 +540,28 @@ public class ImeTest extends ContentShellTestBase {
         assertEquals(KeyEvent.KEYCODE_H, mImeAdapter.mLastSyntheticKeyCode);
     }
 
+    @SmallTest
+    @Feature({"TextInput"})
+    public void testPastePopupShowOnLongPress() throws Throwable {
+        commitText(mConnection, "hello", 1);
+        waitAndVerifyEditableCallback(
+            mConnection.mImeUpdateQueue, 1, "hello", 5, 5, -1, -1);
+
+        selectAll(mImeAdapter);
+        waitAndVerifyEditableCallback(
+            mConnection.mImeUpdateQueue, 2, "hello", 0, 5, -1, -1);
+
+        cut(mImeAdapter);
+        waitAndVerifyEditableCallback(
+            mConnection.mImeUpdateQueue, 0, "", 0, 0, -1, -1);
+
+        DOMUtils.focusNode(mContentViewCore, "input_radio");
+        DOMUtils.longPressNode(this, mContentViewCore, "input_text");
+        PastePopupMenu pastePopup = mContentViewCore.getPastePopupForTest();
+        assertWaitForKeyboardStatus(false);

[jdduke (slow), 2014/08/26 15:06:43]

I don't think this is the right wait condition. The keyboard should be hidden by
the |cut| operation, right? So how does this guarantee we'll have gotten the
long press response?

[raghu, 2014/08/26 15:45:56]

Cut doesnt hide the keyboard. Also in this test since we are long pressing on an
empty input, keyboard status doesnt change at all.
Please suggest me here.

[jdduke (slow), 2014/08/26 16:01:22]

If the keyboard status doesn't change, why are you using it as a wait condition?
Rather than waiting on an external condition, why not simply wait until
pastePopup.isShowing() is true:

final PastePopupMenu pastePopup = mContentViewCore.getPastePopupForTest();
assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
    @Override
    public boolean isSatisfied() {
        return pastePopup.isShowing();
    }
}));

[raghu, 2014/08/27 05:06:42]

Done.

+        assertTrue(pastePopup.isShowing());
+    }
+
     private void performGo(final AdapterInputConnection inputConnection,
             TestCallbackHelperContainer testCallbackHelperContainer) throws Throwable {
         handleBlockingCallbackAction(