From 46d168fb6b50f0bb093f5c0428c8287f2e8d2d08 Mon Sep 17 00:00:00 2001 From: Todd Kennedy Date: Wed, 13 May 2015 11:13:58 -0700 Subject: [PATCH] Don't negate saving state In the original implementation, we would only save the state if the activity was "!isFinishing()". That was abstracted into a method that returned whether we should save state. However, the "!" wasn't removed. D'oh. Bug: 20860094 Change-Id: I8827cb265f2e7fb911f5d110c56e464aa092bfe7 --- core/java/android/app/FragmentManager.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/java/android/app/FragmentManager.java b/core/java/android/app/FragmentManager.java index 6b5239d26ee1b..e0a30ad9c6e06 100644 --- a/core/java/android/app/FragmentManager.java +++ b/core/java/android/app/FragmentManager.java @@ -945,7 +945,7 @@ final class FragmentManagerImpl extends FragmentManager implements LayoutInflate if (f.mView != null) { // Need to save the current view state if not // done already. - if (!mHost.onShouldSaveFragmentState(f) && f.mSavedViewState == null) { + if (mHost.onShouldSaveFragmentState(f) && f.mSavedViewState == null) { saveFragmentViewState(f); } }