From 72843c13b616aa1f0852205f03befacdc43e95aa Mon Sep 17 00:00:00 2001 From: Daichi Hirono Date: Tue, 15 Mar 2016 12:47:57 +0900 Subject: [PATCH] Move to downloads root instead of home root. Previously we assume the home root exists always, but while migrating internal storage, the home root is temporary removed from Android. Since we open donwnloads root by default now, the CL lets Files app move to the downloads directory instead of home root when the current directory is removed. BUG=27570929 Change-Id: I89efb1ae32ba8ae7269b3242035d5b67114fcb54 --- .../com/android/documentsui/BaseActivity.java | 24 +++++++++---------- 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/packages/DocumentsUI/src/com/android/documentsui/BaseActivity.java b/packages/DocumentsUI/src/com/android/documentsui/BaseActivity.java index 1a8ce18ce03d1..c51cbb3b723a8 100644 --- a/packages/DocumentsUI/src/com/android/documentsui/BaseActivity.java +++ b/packages/DocumentsUI/src/com/android/documentsui/BaseActivity.java @@ -670,7 +670,7 @@ public abstract class BaseActivity extends Activity private static final class HandleRootsChangedTask extends PairedTask { - DocumentInfo mHome; + DocumentInfo mDownloadsDocument; public HandleRootsChangedTask(BaseActivity activity) { super(activity); @@ -682,28 +682,28 @@ public abstract class BaseActivity extends Activity final RootInfo currentRoot = roots[0]; final Collection cachedRoots = mOwner.mRoots.getRootsBlocking(); - RootInfo homeRoot = null; + RootInfo downloadsRoot = null; for (final RootInfo root : cachedRoots) { - if (root.isHome()) { - homeRoot = root; + if (root.isDownloads()) { + downloadsRoot = root; } if (root.getUri().equals(currentRoot.getUri())) { // We don't need to change the current root as the current root was not removed. return null; } } - assert(homeRoot != null); - mHome = mOwner.getRootDocumentBlocking(homeRoot); - return homeRoot; + assert(downloadsRoot != null); + mDownloadsDocument = mOwner.getRootDocumentBlocking(downloadsRoot); + return downloadsRoot; } @Override - protected void finish(RootInfo homeRoot) { - if (homeRoot != null && mHome != null) { + protected void finish(RootInfo downloadsRoot) { + if (downloadsRoot != null && mDownloadsDocument != null) { // Clear entire backstack and start in new root - mOwner.mState.onRootChanged(homeRoot); - mOwner.mSearchManager.update(homeRoot); - mOwner.openContainerDocument(mHome); + mOwner.mState.onRootChanged(downloadsRoot); + mOwner.mSearchManager.update(downloadsRoot); + mOwner.openContainerDocument(mDownloadsDocument); } } }