Set the Insertion handle correctly while showing the paste popup menu

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

 // Copyright 2013 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.
 
 package org.chromium.content.browser.input;
 
 import android.app.Activity;
 import android.content.ClipData;
 import android.content.ClipboardManager;
 import android.content.Context;
@@ skipping 532 matching lines @@
 
         // Simulate switch of input fields.
         finishComposingText(mConnection);
 
         // H
         expectUpdateStateCall(mConnection);
         setComposingText(mConnection, "h", 1);
         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.longPressNode(this, mContentViewCore, "input_text");
+

[jdduke (slow), 2014/08/27 17:05:31]

Nit: Adding line breaks here doesn't really help, I would put the longPressNode,
pastePopup assignment and asserTrue lines all next to each other.

[raghu, 2014/08/27 18:45:11]

will do the change

[raghu, 2014/08/28 02:26:16]

Done.

+        final PastePopupMenu pastePopup = mContentViewCore.getPastePopupForTest();
+
+        assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
+            @Override
+            public boolean isSatisfied() {
+                return pastePopup.isShowing();
+            }
+        }));
+    }
+
     private void performGo(final AdapterInputConnection inputConnection,
             TestCallbackHelperContainer testCallbackHelperContainer) throws Throwable {
         handleBlockingCallbackAction(
                 testCallbackHelperContainer.getOnPageFinishedHelper(),
                 new Runnable() {
                     @Override
                     public void run() {
                         inputConnection.performEditorAction(EditorInfo.IME_ACTION_GO);
                     }
                 });
@@ skipping 227 matching lines @@
         public void assertEqualState(String text, int selectionStart, int selectionEnd,
                 int compositionStart, int compositionEnd) {
             assertEquals("Text did not match", text, mText);
             assertEquals("Selection start did not match", selectionStart, mSelectionStart);
             assertEquals("Selection end did not match", selectionEnd, mSelectionEnd);
             assertEquals("Composition start did not match", compositionStart, mCompositionStart);
             assertEquals("Composition end did not match", compositionEnd, mCompositionEnd);
         }
     }
 }