Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-master-dev
commit 5a34c3a6c57ac92342050f3f2d6fa6cf58882597
Author: Jeremy Woods <jbwoods@google.com>
Date: Thu Aug 01 13:16:29 2019
Ensure view is null when StrictViewFragment calls onCreateView
Added check to StrictViewFragment to assert that the view is null before
a Fragment attempt to create anothing one.
Test: all fragment tests pass
BUG:138741697
Change-Id: If1e7c702a9cef4ea2db7d81095eb4cf575e5d9cb
M fragment/fragment/src/androidTest/java/androidx/fragment/app/StrictViewFragment.kt
https://android-review.googlesource.com/1095363
https://goto.google.com/android-sha1/5a34c3a6c57ac92342050f3f2d6fa6cf58882597
Branch: androidx-master-dev
commit 5a34c3a6c57ac92342050f3f2d6fa6cf58882597
Author: Jeremy Woods <jbwoods@google.com>
Date: Thu Aug 01 13:16:29 2019
Ensure view is null when StrictViewFragment calls onCreateView
Added check to StrictViewFragment to assert that the view is null before
a Fragment attempt to create anothing one.
Test: all fragment tests pass
BUG:138741697
Change-Id: If1e7c702a9cef4ea2db7d81095eb4cf575e5d9cb
M fragment/fragment/src/androidTest/java/androidx/fragment/app/StrictViewFragment.kt
jb...@google.com <jb...@google.com>
ap...@google.com <ap...@google.com> #3
Project: platform/frameworks/support
Branch: androidx-master-dev
commit 0942405fb1eb41dd5ff51a7bc807ccdb62ec8224
Author: Jeremy Woods <jbwoods@google.com>
Date: Mon Aug 19 01:39:01 2019
Add tests for mixing transitions with animations and animators
We should always get end calls from both transitions and animations (or
animators) no matter what the durations are.
Test: Added test
BUG: 138741697
Change-Id: I851d5ec4286f9ad97be2c4b42709f6366b8add54
M fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentTestUtil.kt
A fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentTransitionAnimTest.kt
M fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentTransitionTest.kt
M fragment/fragment/src/androidTest/java/androidx/fragment/app/TransitionFragment.kt
https://android-review.googlesource.com/1105166
https://goto.google.com/android-sha1/0942405fb1eb41dd5ff51a7bc807ccdb62ec8224
Branch: androidx-master-dev
commit 0942405fb1eb41dd5ff51a7bc807ccdb62ec8224
Author: Jeremy Woods <jbwoods@google.com>
Date: Mon Aug 19 01:39:01 2019
Add tests for mixing transitions with animations and animators
We should always get end calls from both transitions and animations (or
animators) no matter what the durations are.
Test: Added test
BUG: 138741697
Change-Id: I851d5ec4286f9ad97be2c4b42709f6366b8add54
M fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentTestUtil.kt
A fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentTransitionAnimTest.kt
M fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentTransitionTest.kt
M fragment/fragment/src/androidTest/java/androidx/fragment/app/TransitionFragment.kt
ap...@google.com <ap...@google.com> #4
Project: platform/frameworks/support
Branch: androidx-master-dev
commit a7b3de1d4990e5f306048aa70e6af0240717c671
Author: Jeremy Woods <jbwoods@google.com>
Date: Thu Aug 15 15:24:03 2019
Use ExitAnimationCompleteMarker with Animations and Animators
Animations, Animators, and Transitions are currently all handled
separately by the FragmentManager. Looking toward the future it would be
nice if these and any other animation systems could be handled in the
same (or at least a similar) way by the FragmentManager.
This change uses the ExitAnimationCompleteMarker interface to allow
proper behavior when Animations and Animators are ended or canceled. A
cancel method was added to allow animation components to define how they
should be handled when the FragmentManager needs to cancel an animation
early. When animations are ended, the animation component removes the
ExitAnimationCompleteMarker and that notifies the FragmentManager that
the animation has ended, the view can be destroyed, and the fragment can
be moved to the proper state.
Test: all tests pass
BUG: 138741697
Change-Id: I315b814132a57ab89c0ccd127161b28e60628996
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentTransition.java
https://android-review.googlesource.com/1103911
https://goto.google.com/android-sha1/a7b3de1d4990e5f306048aa70e6af0240717c671
Branch: androidx-master-dev
commit a7b3de1d4990e5f306048aa70e6af0240717c671
Author: Jeremy Woods <jbwoods@google.com>
Date: Thu Aug 15 15:24:03 2019
Use ExitAnimationCompleteMarker with Animations and Animators
Animations, Animators, and Transitions are currently all handled
separately by the FragmentManager. Looking toward the future it would be
nice if these and any other animation systems could be handled in the
same (or at least a similar) way by the FragmentManager.
This change uses the ExitAnimationCompleteMarker interface to allow
proper behavior when Animations and Animators are ended or canceled. A
cancel method was added to allow animation components to define how they
should be handled when the FragmentManager needs to cancel an animation
early. When animations are ended, the animation component removes the
ExitAnimationCompleteMarker and that notifies the FragmentManager that
the animation has ended, the view can be destroyed, and the fragment can
be moved to the proper state.
Test: all tests pass
BUG: 138741697
Change-Id: I315b814132a57ab89c0ccd127161b28e60628996
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentManager.java
M fragment/fragment/src/main/java/androidx/fragment/app/FragmentTransition.java
jb...@google.com <jb...@google.com> #5
This issue has been fixed internally and will be available in the Fragment 1.2.0-alpha03 release.
ap...@google.com <ap...@google.com> #6
Project: platform/frameworks/support
Branch: androidx-master-dev
commit a9bbe1a2a1bf4b6285377352808748888786062a
Author: Jeremy Woods <jbwoods@google.com>
Date: Tue Sep 03 15:09:20 2019
Add test to ensure child fragments call onDestroyView on config change
If a child fragment has animation and the parent does not, the view for
a child fragment should still be destroyed when the parent fragment's view is.
Test: Added Test
BUG: 138741697
Change-Id: I376e59515490220959aaebca5cbd78757887b2c7
M fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentLifecycleTest.kt
https://android-review.googlesource.com/1113331
https://goto.google.com/android-sha1/a9bbe1a2a1bf4b6285377352808748888786062a
Branch: androidx-master-dev
commit a9bbe1a2a1bf4b6285377352808748888786062a
Author: Jeremy Woods <jbwoods@google.com>
Date: Tue Sep 03 15:09:20 2019
Add test to ensure child fragments call onDestroyView on config change
If a child fragment has animation and the parent does not, the view for
a child fragment should still be destroyed when the parent fragment's view is.
Test: Added Test
BUG: 138741697
Change-Id: I376e59515490220959aaebca5cbd78757887b2c7
M fragment/fragment/src/androidTest/java/androidx/fragment/app/FragmentLifecycleTest.kt
Description