drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Null check for restoring LoaderManagers

Browse Source 
am: 100ba76d7a

Change-Id: I7b5eecf24d73fd62abaa2825374bd936f15425e5
This commit is contained in:
Adam Powell
2016-09-22 20:38:00 +00:00
committed by android-build-merger
parent 1197c16ea7 100ba76d7a
commit bb2011a2e0
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
core/java/android/app/FragmentHostCallback.java
View File

@ -340,8 +340,10 @@ public abstract class FragmentHostCallback<E> extends FragmentContainer {
} }
void restoreLoaderNonConfig(ArrayMap<String, LoaderManager> loaderManagers) { void restoreLoaderNonConfig(ArrayMap<String, LoaderManager> loaderManagers) {
for (int i = 0, N = loaderManagers.size(); i < N; i++) { if (loaderManagers != null) {
((LoaderManagerImpl) loaderManagers.valueAt(i)).updateHostController(this); for (int i = 0, N = loaderManagers.size(); i < N; i++) {
((LoaderManagerImpl) loaderManagers.valueAt(i)).updateHostController(this);
}
} }
mAllLoaderManagers = loaderManagers; mAllLoaderManagers = loaderManagers;
} }

3
core/tests/coretests/src/android/app/LoaderLifecycleTest.java
View File

@ -99,6 +99,9 @@ public class LoaderLifecycleTest {
final FragmentManager fm2 = fc2.getFragmentManager(); final FragmentManager fm2 = fc2.getFragmentManager();
fc2.attachHost(null); fc2.attachHost(null);
// Make sure nothing blows up on a null here
fc2.restoreLoaderNonConfig(null);
// for real this time
fc2.restoreLoaderNonConfig(loaderNonConfig); fc2.restoreLoaderNonConfig(loaderNonConfig);
fc2.restoreAllState(savedState, nonconf); fc2.restoreAllState(savedState, nonconf);
fc2.dispatchCreate(); fc2.dispatchCreate();