Change theme
Help
Press space for more information.
Show links for this issue (Shortcut: i, l)
Copy issue ID
Previous Issue (Shortcut: k)
Next Issue (Shortcut: j)
Sign in to use full features.
Vote: I am impacted
Notification menu
Refresh (Shortcut: Shift+r)
Go home (Shortcut: u)
Pending code changes (auto-populated)
View issue level access limits(Press Alt + Right arrow for more information)
Unintended behavior
View staffing
Description
With the currently implementation of
visibleTransitionsInProgress
in possible that there is a scenario where you can have duplicate entries in the list and cause the wrong back stack entry to be composed.This is because we first add all of the
STARTED
entries to the lists and then we add aLifecycleObserver
than then re-adds entries when they go throughON_START
.While we could change the list to a
MutableSet
to avoid this, we would then lose the snapshot functionality ofmutableStateListOf()
and we want to still recompose in response to adding and removing entries from the list. So we should just treat the list like it is a set manually instead.