commit b20ed181e496c1cc8c36f6eacf3a60a9569ee0fb
author Ivan Chiang <chiangi@google.com> Wed Nov 28 00:55:25 2018 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Wed Nov 28 00:55:25 2018 +0000
tree ce906157206dfb7bb1ecd2659c01326b6fc4ff9a
parent f12a1078495d896ba0a995410c87934925002bab
parent f3448693688b506f3ec91ad409cd09ee0355f6c8

Merge "Use the query bundle to search"

src/com/android/documentsui/DirectoryLoader.java

diff --git a/src/com/android/documentsui/DirectoryLoader.java b/src/com/android/documentsui/DirectoryLoader.java
index 1750dfa..233d4d1 100644
--- a/src/com/android/documentsui/DirectoryLoader.java
+++ b/src/com/android/documentsui/DirectoryLoader.java
@@ -31,6 +31,7 @@
 import android.os.Looper;
 import android.os.OperationCanceledException;
 import android.os.RemoteException;
+import android.provider.DocumentsContract;
 import android.provider.DocumentsContract.Document;
 import android.util.Log;
 
@@ -121,20 +122,24 @@
             result.client = client;
 
             Resources resources = getContext().getResources();
-            if (mFeatures.isContentPagingEnabled()) {
-                Bundle queryArgs = new Bundle();
-                mModel.addQuerySortArgs(queryArgs);
 
+            final Bundle queryArgs = new Bundle();
+            mModel.addQuerySortArgs(queryArgs);
+
+            if (mSearchMode) {
+                // add the search string into the query bundle.
+                queryArgs.putString(DocumentsContract.QUERY_ARG_DISPLAY_NAME,
+                        DocumentsContract.getSearchDocumentsQuery(mUri));
+            }
+
+            if (mFeatures.isContentPagingEnabled()) {
                 // TODO: At some point we don't want forced flags to override real paging...
                 // and that point is when we have real paging.
                 DebugFlags.addForcedPagingArgs(queryArgs);
-
-                cursor = client.query(mUri, null, queryArgs, mSignal);
-            } else {
-                cursor = client.query(
-                        mUri, null, null, null, mModel.getDocumentSortQuery(), mSignal);
             }
 
+            cursor = client.query(mUri, null, queryArgs, mSignal);
+
             if (cursor == null) {
                 throw new RemoteException("Provider returned null");
             }
@@ -151,7 +156,7 @@
             // TODO: When API tweaks have landed, use ContentResolver.EXTRA_HONORED_ARGS
             // instead of checking directly for ContentResolver.QUERY_ARG_SORT_COLUMNS (won't work)
             if (mFeatures.isContentPagingEnabled()
-                        && cursor.getExtras().containsKey(ContentResolver.QUERY_ARG_SORT_COLUMNS)) {
+                    && cursor.getExtras().containsKey(ContentResolver.QUERY_ARG_SORT_COLUMNS)) {
                 if (VERBOSE) Log.d(TAG, "Skipping sort of pre-sorted cursor. Booya!");
             } else {
                 cursor = mModel.sortCursor(cursor, mFileTypeLookup);