IssueTracker

Tram, can you confirm that the behavior is correct if the modifier is moved to the composable inside the AnimatedPane?

Hi Max, yes when the modifiers are moved to the composable inside the AnimatedPane, the PB animations are better than when the modifiers are in the AnimatedPane itself

Given the straightforward workaround in comment#3, I'm going to downgrade the priority of this bug for now

After some investigation, I can conclude this behavior is consistent with other animation "wrapper" composables, e.g. AnimatedVisibility. Modifiers should indeed be applied to the children in order to properly participate in the animation.

The regression range says this regressed last Nov, which would rule out scoped view transitions. But none of the CLs in the range look plausible.

Scoped view transitions is behind a flag (status:experimental) and doesn't affect the lifecycle of document-level view transitions.

Tentatively reassigning back to vmpstr@.

I think the problem here is that although the testcase has no vt, it triggers during a gc sweep which may have previous test information in it. And yeah, I'm not sure if the regression range is accurate since we should have seen this issue all along. But this is a beta blocker, which presumably means it's a recent regression

This issue is marked as a release blocker with no milestone associated. Please add an appropriate milestone.

All release blocking issues should have milestones associated to it, so that the issue can tracked and the fixes can be pushed promptly.

Thanks for your time! To disable nags, add Disable-Nags (case sensitive) to the Chromium Labels custom field.

Project: chromium/src
Branch: main
Author: Vladimir Levin <vmpstr@chromium.org>
Link:      https://chromium-review.googlesource.com/6330377

vt: Allow kIdle state when destroying VT style tracker

vt: Allow kIdle state when destroying VT style tracker 
 
Since the style tracker can be destroyed before kCapturing phase 
(but after kInitial), we need to also allow the kIdle state for the 
style tracker. 
 
R=khushalsagar@chromium.org 
 
Bug: 399698557 
Change-Id: I3e738fe172dd04735b4fbcf5ebe97f32dd883c01 
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6330377 
Commit-Queue: Vladimir Levin <vmpstr@chromium.org> 
Reviewed-by: Khushal Sagar <khushalsagar@chromium.org> 
Cr-Commit-Position: refs/heads/main@{#1428964}

This should be fixed in #9. If possible, please verify