diff options
author | 2018-11-06 14:39:40 +0800 | |
---|---|---|
committer | 2018-11-06 17:28:36 +0800 | |
commit | 35ad6bd641820432172feb50875108f0b93ee8ce (patch) | |
tree | 5d0dc6247fe9b73195e116414738c0ce29f3bc1f | |
parent | 402ad09962268b9d1f18c28151a2f57bf586b75b (diff) |
Update all loader to androidx version loader
Due to aosp/810773 megerd, it can fully update to androix loader.
Bug: 111921428
Test: atest DocumentsUITests
Change-Id: I9cd44790717f97db154b315797de261ff8d91c7d
4 files changed, 15 insertions, 7 deletions
diff --git a/src/com/android/documentsui/AbstractActionHandler.java b/src/com/android/documentsui/AbstractActionHandler.java index a95b1ec9f..828e14ea1 100644 --- a/src/com/android/documentsui/AbstractActionHandler.java +++ b/src/com/android/documentsui/AbstractActionHandler.java @@ -20,12 +20,10 @@ import static com.android.documentsui.base.DocumentInfo.getCursorInt; import static com.android.documentsui.base.DocumentInfo.getCursorString; import static com.android.documentsui.base.SharedMinimal.DEBUG; -import android.app.LoaderManager.LoaderCallbacks; import android.app.PendingIntent; import android.content.Context; import android.content.Intent; import android.content.IntentSender; -import android.content.Loader; import android.content.pm.ResolveInfo; import android.database.Cursor; import android.graphics.drawable.ColorDrawable; @@ -39,6 +37,8 @@ import android.view.DragEvent; import androidx.annotation.VisibleForTesting; import androidx.fragment.app.FragmentActivity; +import androidx.loader.app.LoaderManager.LoaderCallbacks; +import androidx.loader.content.Loader; import androidx.recyclerview.selection.ItemDetailsLookup.ItemDetails; import androidx.recyclerview.selection.MutableSelection; import androidx.recyclerview.selection.SelectionTracker; @@ -500,7 +500,7 @@ public abstract class AbstractActionHandler<T extends FragmentActivity & CommonA return; } - mActivity.getLoaderManager().restartLoader(LOADER_ID, null, mBindings); + mActivity.getSupportLoaderManager().restartLoader(LOADER_ID, null, mBindings); } protected final boolean launchToDocument(Uri uri) { diff --git a/src/com/android/documentsui/DirectoryLoader.java b/src/com/android/documentsui/DirectoryLoader.java index 0a749fb65..1750dfa9e 100644 --- a/src/com/android/documentsui/DirectoryLoader.java +++ b/src/com/android/documentsui/DirectoryLoader.java @@ -18,7 +18,6 @@ package com.android.documentsui; import static com.android.documentsui.base.SharedMinimal.VERBOSE; -import android.content.AsyncTaskLoader; import android.content.ContentProviderClient; import android.content.ContentResolver; import android.content.Context; @@ -47,6 +46,10 @@ import com.android.documentsui.sorting.SortModel; import android.os.FileUtils; +import androidx.loader.content.AsyncTaskLoader; + +import java.util.concurrent.Executor; + public class DirectoryLoader extends AsyncTaskLoader<DirectoryResult> { private static final String TAG = "DirectoryLoader"; @@ -77,7 +80,7 @@ public class DirectoryLoader extends AsyncTaskLoader<DirectoryResult> { ContentLock lock, boolean inSearchMode) { - super(context, ProviderExecutor.forAuthority(root.authority)); + super(context); mFeatures = features; mRoot = root; mUri = uri; @@ -89,6 +92,11 @@ public class DirectoryLoader extends AsyncTaskLoader<DirectoryResult> { } @Override + protected Executor getExecutor() { + return ProviderExecutor.forAuthority(mRoot.authority); + } + + @Override public final DirectoryResult loadInBackground() { synchronized (this) { if (isLoadInBackgroundCanceled()) { diff --git a/src/com/android/documentsui/RecentsLoader.java b/src/com/android/documentsui/RecentsLoader.java index 42be08294..0db270d96 100644 --- a/src/com/android/documentsui/RecentsLoader.java +++ b/src/com/android/documentsui/RecentsLoader.java @@ -20,7 +20,6 @@ import static com.android.documentsui.base.SharedMinimal.DEBUG; import static com.android.documentsui.base.SharedMinimal.TAG; import android.app.ActivityManager; -import android.content.AsyncTaskLoader; import android.content.ContentProviderClient; import android.content.Context; import android.database.Cursor; @@ -42,6 +41,7 @@ import com.android.documentsui.base.State; import com.android.documentsui.roots.ProvidersAccess; import com.android.documentsui.roots.RootCursorWrapper; import androidx.annotation.GuardedBy; +import androidx.loader.content.AsyncTaskLoader; import com.google.common.util.concurrent.AbstractFuture; diff --git a/tests/unit/com/android/documentsui/AbstractActionHandlerTest.java b/tests/unit/com/android/documentsui/AbstractActionHandlerTest.java index afb895251..cb18279ad 100644 --- a/tests/unit/com/android/documentsui/AbstractActionHandlerTest.java +++ b/tests/unit/com/android/documentsui/AbstractActionHandlerTest.java @@ -261,7 +261,7 @@ public class AbstractActionHandlerTest { mHandler.loadDocumentsForCurrentStack(); CountDownLatch latch = new CountDownLatch(1); mEnv.model.addUpdateListener(event -> latch.countDown()); - mActivity.loaderManager.runAsyncTaskLoader(AbstractActionHandler.LOADER_ID); + mActivity.supportLoaderManager.runAsyncTaskLoader(AbstractActionHandler.LOADER_ID); latch.await(1, TimeUnit.SECONDS); assertEquals(2, mEnv.model.getItemCount()); |