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

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

Issue 2932433002: Photo Picker dialog: Pipe through the MIME type selections from the web page. (Closed)
Patch Set: 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/photo_picker/PhotoPickerDialogTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/photo_picker/PhotoPickerDialogTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/photo_picker/PhotoPickerDialogTest.java
index 83cc376f38c36dc6a8a38cb951a4111f92662cdd..6d74b14bd45a430b2d2c973f0d8814bb8efa136e 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/photo_picker/PhotoPickerDialogTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/photo_picker/PhotoPickerDialogTest.java
@@ -103,13 +103,14 @@ public class PhotoPickerDialogTest extends ChromeActivityTestCaseBase<ChromeActi
return (RecyclerView) mDialog.findViewById(R.id.recycler_view);
}
- private PhotoPickerDialog createDialog(final boolean multiselect) throws Exception {
+ private PhotoPickerDialog createDialog(final boolean multiselect, final List<String> mimeTypes)
+ throws Exception {
final PhotoPickerDialog dialog =
ThreadUtils.runOnUiThreadBlocking(new Callable<PhotoPickerDialog>() {
@Override
public PhotoPickerDialog call() {
final PhotoPickerDialog dialog = new PhotoPickerDialog(
- getActivity(), PhotoPickerDialogTest.this, multiselect);
+ getActivity(), PhotoPickerDialogTest.this, multiselect, mimeTypes);
dialog.show();
return dialog;
}
@@ -170,7 +171,7 @@ public class PhotoPickerDialogTest extends ChromeActivityTestCaseBase<ChromeActi
@LargeTest
public void testNoSelection() throws Throwable {
- createDialog(false); // Multi-select = false.
+ createDialog(false, Arrays.asList("image/*")); // Multi-select = false.
assertTrue(mDialog.isShowing());
int expectedSelectionCount = 1;
@@ -185,7 +186,7 @@ public class PhotoPickerDialogTest extends ChromeActivityTestCaseBase<ChromeActi
@LargeTest
public void testSingleSelectionPhoto() throws Throwable {
- createDialog(false); // Multi-select = false.
+ createDialog(false, Arrays.asList("image/*")); // Multi-select = false.
assertTrue(mDialog.isShowing());
// Expected selection count is 1 because clicking on a new view unselects other.
@@ -203,7 +204,7 @@ public class PhotoPickerDialogTest extends ChromeActivityTestCaseBase<ChromeActi
@LargeTest
public void testMultiSelectionPhoto() throws Throwable {
- createDialog(true); // Multi-select = true.
+ createDialog(true, Arrays.asList("image/*")); // Multi-select = true.
assertTrue(mDialog.isShowing());
// Multi-selection is enabled, so each click is counted.

Powered by Google App Engine
This is Rietveld 408576698