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

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

Issue 2775393002: Add multi line to Tabular Context Menu (Closed)
Patch Set: git rebase 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 | « chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUi.java ('k') | no next file » | 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/contextmenu/TabularContextMenuUiTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUiTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUiTest.java
index 97229964c33f0daa7f4a437a5888e03dca8834dc..745fd5d0123c3f4f30cb523008910bb0d91607af 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUiTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUiTest.java
@@ -139,4 +139,38 @@ public class TabularContextMenuUiTest extends ChromeActivityTestCaseBase<ChromeA
assertEquals(view.findViewById(R.id.context_header_text).getVisibility(), View.GONE);
assertEquals(view.findViewById(R.id.context_divider).getVisibility(), View.GONE);
}
+
+ @SmallTest
+ @Feature({"CustomContextMenu"})
+ public void testLinkShowsMultipleLinesWhenClicked() throws ExecutionException {
+ final TabularContextMenuUi dialog = new TabularContextMenuUi();
+ final List<ContextMenuItem> item =
+ CollectionUtil.newArrayList(ContextMenuItem.ADD_TO_CONTACTS, ContextMenuItem.CALL,
+ ContextMenuItem.COPY_LINK_ADDRESS);
+ View view = ThreadUtils.runOnUiThreadBlocking(new Callable<View>() {
+ @Override
+ public View call() {
+ return dialog.createContextMenuPageUi(
+ getActivity(), new MockMenuParams("http://google.com"), item, item.size());
+ }
+ });
+
+ final TextView headerTextView = (TextView) view.findViewById(R.id.context_header_text);
+ int expectedMaxLines = 1;
+ int actualMaxLines = headerTextView.getMaxLines();
+ assertEquals("Expected a different number of default maximum lines.", expectedMaxLines,
+ actualMaxLines);
+
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ headerTextView.callOnClick();
+ }
+ });
+
+ expectedMaxLines = Integer.MAX_VALUE;
+ actualMaxLines = headerTextView.getMaxLines();
+ assertEquals("Expected a different number of maximum lines when the header is clicked.",
+ expectedMaxLines, actualMaxLines);
+ }
}
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/contextmenu/TabularContextMenuUi.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698