Bug 191700 – Regression: Layer doesn't move when transform updated after a transition

• Status: RESOLVED
• Resolution: CONFIGURATION CHANGED
• Component: Animations

[Ali Juma]

Created attachment 354968 [details]
Test case

See attached test case: the blue box is animated using a CSS transition, and then its transform is updated again after the transition finishes. But the layer doesn't move for the latter update, until an update is forced by resizing the window and/or minimizing and restoring it.

This works in STP 69 but is broken on trunk.

Trying to debug this a bit, we do get as far as setting the new transform on CALayer, but the CALayer still has an animation on it (even though the transition is finished) and ignores the new transform. If that animation is removed before setting the transform, the layer does move to the right place.

[Simon Fraser (smfr)]

Almost certainly a regression from my compositing changes.

[Simon Fraser (smfr)]

I can't reproduce. Are you sure this wasn't fixed by r238170?

[Frédéric Wang (:fredw)]

I can only reproduce this issue on the iOS simulator, not macOS safari/minibrowser. I'm at r238191

[Ali Juma]

I tried again at r238277, and this still reproduces for me on macOS MiniBrowser and Safari.

[Simon Fraser (smfr)]

I can reproduce in the simulator.

[Simon Fraser (smfr)]

It seems like we're failing to remove the transforms from the GraphicsLayer.

[Simon Fraser (smfr)]

I think this is an animations bug. We don't hit KeyframeEffect::applyPendingAcceleratedActions() when the transition ends.

[Simon Fraser (smfr)]

https://trac.webkit.org/changeset/238128/webkit may be related.

[Radar WebKit Bug Importer]

<rdar://problem/46130742>

[Antoine Quint]

Does not reproduce with ToT for me.