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

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

Issue 2942963002: corrected wrong options showing on context menus. (Closed)
Patch Set: empty Created 3 years, 6 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
Index: chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
index 7d0f8e04f532fa9daab0f5e0b247cae800e01539..14fd9e8f322dd7a4e57960fc9e51f1f41e186c1b 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
@@ -137,33 +137,6 @@ public class ContextMenuTest implements CustomMainActivityStart {
assertStringContains("test_link.html", getClipboardText());
}
- @Test
- @MediumTest
- @Feature({"Browser"})
- @RetryOnFailure
- public void testCopyLinkTextSimple() throws Throwable {
- Tab tab = mDownloadTestRule.getActivity().getActivityTab();
- ContextMenuUtils.selectContextMenuItem(InstrumentationRegistry.getInstrumentation(),
- mDownloadTestRule.getActivity(), tab, "testLink", R.id.contextmenu_copy_link_text);
-
- Assert.assertEquals(
- "Clipboard text was not what was expected", "Test Link", getClipboardText());
- }
-
- @Test
- @MediumTest
- @Feature({"Browser"})
- @RetryOnFailure
- public void testCopyLinkTextComplex() throws Throwable {
- Tab tab = mDownloadTestRule.getActivity().getActivityTab();
- ContextMenuUtils.selectContextMenuItem(InstrumentationRegistry.getInstrumentation(),
- mDownloadTestRule.getActivity(), tab, "copyLinkTextComplex",
- R.id.contextmenu_copy_link_text);
-
- Assert.assertEquals("Clipboard text was not what was expected",
- "This is pretty extreme \n(newline). ", getClipboardText());
- }
-
@Test
@MediumTest
@Feature({"Browser"})
@@ -399,8 +372,8 @@ public class ContextMenuTest implements CustomMainActivityStart {
ContextMenu menu = ContextMenuUtils.openContextMenu(tab, "testLink");
Integer[] expectedItems = {R.id.contextmenu_open_in_new_tab,
- R.id.contextmenu_open_in_incognito_tab, R.id.contextmenu_copy_link_address,
- R.id.contextmenu_copy_link_text, R.id.contextmenu_save_link_as};
+ R.id.contextmenu_open_in_incognito_tab, R.id.contextmenu_save_link_as,
+ R.id.contextmenu_copy_link_address, R.id.contextmenu_share_link};
assertMenuItemsAreEqual(menu, expectedItems);
}
@@ -432,7 +405,7 @@ public class ContextMenuTest implements CustomMainActivityStart {
R.id.contextmenu_open_in_incognito_tab, R.id.contextmenu_copy_link_address,
R.id.contextmenu_save_link_as, R.id.contextmenu_save_image,
R.id.contextmenu_open_image_in_new_tab, R.id.contextmenu_search_by_image,
- R.id.contextmenu_share_image};
+ R.id.contextmenu_share_image, R.id.contextmenu_share_link};
assertMenuItemsAreEqual(menu, expectedItems);
}

Powered by Google App Engine
This is Rietveld 408576698