Lifecycle should not allow moving from INITIALIZED to DESTROYED [177924329]

Fixed

Bug

Status Update

No update yet.

Description

jb...@google.com

created issue #1

Jan 19, 2021 08:04PM

If you add an observers to a Lifecycle object and attempt to move that Lifecycle.State from INITIALIZED to DESTROYED, the following error will be thrown:

java.lang.IllegalStateException: no event down from INITIALIZED

If there is no observer, then the lifecycle state is moved without any error.

Moving the Lifecycle.State from INITIALIZED to DESTROYED should be consistent whether or not there is an observer on the Lifecycle.

Comments

il...@google.com <il...@google.com> Nov 18, 2021 10:12PM

Status: New

jb...@google.com <jb...@google.com> Feb 4, 2022 12:34AM

Assigned to jb...@google.com.

ap...@google.com <ap...@google.com> #2Feb 7, 2022 02:27PM

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> #3Feb 7, 2022 06:05PM

Marked as fixed.

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().

ad...@gmail.com <ad...@gmail.com> #4Feb 28, 2022 03:11PM

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).

l....@gmail.com <l....@gmail.com> #5Mar 31, 2022 09:02AM

@ #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.

il...@google.com <il...@google.com> #6Mar 31, 2022 11:28PM

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.