Lifecycle should not allow moving from DESTROYED [370577987]

Fixed

Bug

Status Update

No update yet.

Description

jb...@google.com

created issue #1

Sep 30, 2024 10:55PM

b/177924329 addressed an issue of moving the Lifecycle.State from INITIALIZED to DESTROYED needing to throw whether there was an observer or not.

But attempting to move the Lifecycle.State from DESTROYED to any other state should throw as well as DESTROYED is a terminal state for any component that has a Lifecycle.

Comments

ap...@google.com <ap...@google.com> #2Oct 16, 2024 09:32AM

Jeremy, is this still an issue? I think the problem was that you had two transitions targeting the same View for the same action (e.g. two Slide() transitions).

il...@google.com <il...@google.com> Oct 16, 2024 05:24PM

Assigned to mg...@google.com.

mg...@google.com <mg...@google.com> #3Oct 17, 2024 10:38AM

I have a similar issue with plain AnimatorSet:

set.start()
set.pause()
set.setCurrentPlayTime(100)
set.setCurrentPlayTime(0)
set.setCurrentPlayTime(100)
set.resume()

doesn't play animation in resume().

ap...@google.com <ap...@google.com> #4Oct 17, 2024 05:13PM

Should clarify that if I filter out setCurrentPlayTime(0) (or replace it with setCurrentPlayTime(1)) it works well.

Also even with setCurrentPlayTime(0), onAnimationEnd is notified with correct delay (as if the animation has played).

mg...@google.com <mg...@google.com> Oct 24, 2024 03:36PM

Marked as fixed.

il...@google.com <il...@google.com> #5Oct 25, 2024 05:28AM

@ #4:

I think that is intended for Animator. If you set the currentPlayTime to 0 or the total duration the animator completes. We do some special checks in fragments to handle it.

pr...@google.com <pr...@google.com> #6Oct 30, 2024 05:37PM

Did some investigation on the Fragment side and it seems like the merged transition is targeting correctly.

Exiting Transition: Slide@aa9288e: tgts(android.widget.LinearLayout{f9add3d})
>>>>> ExitingViews <<<<<
View: android.widget.LinearLayout{f9add3d}
Entering Transition: Slide@35b8af: tgts(android.widget.LinearLayout{b7f24bc})
>>>>> EnteringViews <<<<<
View: android.widget.LinearLayout{b7f24bc}
Final merged transition: TransitionSet@7bc1c45: 
      TransitionSet@e133f9a: 
        Slide@aa9288e: tgts(android.widget.LinearLayout{f9add3d})
      Slide@35b8af: tgts(android.widget.LinearLayout{b7f24bc})
merged transition passed to controlDelayedTransition: TransitionSet@7bc1c45: 
      TransitionSet@e133f9a: 
        Slide@aa9288e: tgts(android.widget.LinearLayout{f9add3d})
      Slide@35b8af: tgts(android.widget.LinearLayout{b7f24bc})

Still digging.