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

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

Issue 2640333003: [Downloads UI] Implement search (Closed)
Patch Set: Explicitly handle empty downloads query Created 3 years, 11 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/download/DownloadActivityTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadActivityTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadActivityTest.java
index db7d9fba25d141733bb847c8ddfe5378034ab629..c094180824da6f46c44c95d1fe5545c49dbaac2e 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadActivityTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadActivityTest.java
@@ -25,6 +25,7 @@ import org.chromium.chrome.browser.download.ui.DownloadHistoryAdapter;
import org.chromium.chrome.browser.download.ui.DownloadHistoryItemViewHolder;
import org.chromium.chrome.browser.download.ui.DownloadHistoryItemWrapper;
import org.chromium.chrome.browser.download.ui.DownloadItemView;
+import org.chromium.chrome.browser.download.ui.DownloadManagerToolbar;
import org.chromium.chrome.browser.download.ui.DownloadManagerUi;
import org.chromium.chrome.browser.download.ui.SpaceDisplay;
import org.chromium.chrome.browser.download.ui.StubbedProvider;
@@ -538,6 +539,48 @@ public class DownloadActivityTest extends BaseActivityInstrumentationTestCase<Do
assertFalse(mStubbedProvider.getSelectionDelegate().isSelectionEnabled());
}
+ @MediumTest
+ public void testSearchView() throws Exception {
+ final DownloadManagerToolbar toolbar = mUi.getDownloadManagerToolbarForTests();
+ View toolbarSearchView = toolbar.getSearchViewForTests();
+ assertEquals(View.GONE, toolbarSearchView.getVisibility());
+
+ toggleItemSelection(2);
+ assertTrue(mStubbedProvider.getSelectionDelegate().isSelectionEnabled());
+
+ int callCount = mAdapterObserver.onSelectionCallback.getCallCount();
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ toolbar.getMenu().performIdentifierAction(R.id.search_menu_id, 0);
+ }
+ });
+
+ // The selection should be cleared when a search is started.
+ mAdapterObserver.onSelectionCallback.waitForCallback(callCount, 1);
+ assertFalse(mStubbedProvider.getSelectionDelegate().isSelectionEnabled());
+ assertEquals(View.VISIBLE, toolbarSearchView.getVisibility());
+
+ // Select an item and assert that the search view is no longer showing.
+ toggleItemSelection(2);
+ assertTrue(mStubbedProvider.getSelectionDelegate().isSelectionEnabled());
+ assertEquals(View.GONE, toolbarSearchView.getVisibility());
+
+ // Clear the selection and assert that the search view is showing again.
+ toggleItemSelection(2);
+ assertFalse(mStubbedProvider.getSelectionDelegate().isSelectionEnabled());
+ assertEquals(View.VISIBLE, toolbarSearchView.getVisibility());
+
+ // Close the search view.
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ toolbar.onNavigationBack();
+ }
+ });
+ assertEquals(View.GONE, toolbarSearchView.getVisibility());
+ }
+
private DownloadActivity startDownloadActivity() throws Exception {
// Load up the downloads lists.
DownloadItem item0 = StubbedProvider.createDownloadItem(0, "19551112 06:38");

Powered by Google App Engine
This is Rietveld 408576698