Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1274)

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java

Issue 2734943005: Remove moveCursorToSelectionEnd() (Closed)
Patch Set: For dtrainor@'s review Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchManagerTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
index 390137ea2e62b4a714d64c7a7e4e664b2d7daf65..bb31280ec3408791eaf0029435339c4c6a88f6fb 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java
@@ -8,6 +8,7 @@ import static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_NON_LOW_E
import android.content.DialogInterface;
import android.content.pm.ActivityInfo;
+import android.graphics.Point;
import android.os.Debug;
import android.os.SystemClock;
import android.support.test.filters.LargeTest;
@@ -352,6 +353,62 @@ public class TabsTest extends ChromeTabbedActivityTestBase {
assertWaitForKeyboardStatus(false);
}
+ private void assertWaitForSelectedText(final String text) throws InterruptedException {
+ CriteriaHelper.pollUiThread(new Criteria() {
+ @Override
+ public boolean isSatisfied() {
+ final String actualText =
+ getActivity().getActivityTab().getContentViewCore().getSelectedText();
+ updateFailureReason(
+ "expected selected text: [" + text + "], but got: [" + actualText + "]");
+ return text.equals(actualText);
+ }
+ });
+ }
+
+ /**
+ * Generate a fling sequence from the given start/end X,Y percentages, for the given steps.
+ * Works in either landscape or portrait orientation.
+ */
+ private void fling(float startX, float startY, float endX, float endY, int stepCount) {
+ Point size = new Point();
+ getActivity().getWindowManager().getDefaultDisplay().getSize(size);
+ float dragStartX = size.x * startX;
+ float dragEndX = size.x * endX;
+ float dragStartY = size.y * startY;
+ float dragEndY = size.y * endY;
+ long downTime = SystemClock.uptimeMillis();
+ dragStart(dragStartX, dragStartY, downTime);
+ dragTo(dragStartX, dragEndX, dragStartY, dragEndY, stepCount, downTime);
+ dragEnd(dragEndX, dragEndY, downTime);
+ }
+
+ private void scrollDown() {
+ fling(0.f, 0.5f, 0.f, 0.75f, 100);
+ }
+
+ /**
+ * Verify that the selection is collapsed when switching to the tab-switcher mode then switching
+ * back. https://crbug.com/697756
+ */
+ @MediumTest
+ @Restriction(ChromeRestriction.RESTRICTION_TYPE_PHONE)
+ @Feature({"Android-TabSwitcher"})
+ @RetryOnFailure
+ public void testTabSwitcherCollapseSelection() throws Exception {
+ mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
+ ChromeTabUtils.fullyLoadUrlInNewTab(
+ getInstrumentation(), getActivity(), mTestServer.getURL(TEST_FILE_PATH), false);
+ DOMUtils.longPressNode(getActivity().getActivityTab().getContentViewCore(), "textarea");
+ assertWaitForSelectedText("helloworld");
+
+ // Switch to tab-switcher mode, switch back, and scroll page.
+ showOverviewAndWaitForAnimation();
+ hideOverviewAndWaitForAnimation();
+ scrollDown();
+ assertWaitForSelectedText("");
+ }
+
/**
* Verify that opening a new tab and navigating immediately sets a size on the newly created
* renderer. https://crbug.com/434477.
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchManagerTest.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698