Bug 191700

Summary: Regression: Layer doesn't move when transform updated after a transition
Product: WebKit Reporter: Ali Juma <ajuma>
Component: AnimationsAssignee: Antoine Quint <graouts>
Status: RESOLVED CONFIGURATION CHANGED    
Severity: Normal CC: dino, fred.wang, graouts, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case none

Ali Juma
Reported 2018-11-15 12:15:03 PST
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.
Attachments
Test case (723 bytes, text/html)
2018-11-15 12:15 PST, Ali Juma 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2018-11-15 12:36:06 PST
Almost certainly a regression from my compositing changes.
Simon Fraser (smfr)
Comment 2 2018-11-15 21:08:46 PST
I can't reproduce. Are you sure this wasn't fixed by r238170?
Frédéric Wang (:fredw)
Comment 3 2018-11-15 23:15:02 PST
I can only reproduce this issue on the iOS simulator, not macOS safari/minibrowser. I'm at r238191
Ali Juma
Comment 4 2018-11-16 07:34:00 PST
I tried again at r238277, and this still reproduces for me on macOS MiniBrowser and Safari.
Simon Fraser (smfr)
Comment 5 2018-11-16 09:44:48 PST
I can reproduce in the simulator.
Simon Fraser (smfr)
Comment 6 2018-11-16 10:06:14 PST
It seems like we're failing to remove the transforms from the GraphicsLayer.
Simon Fraser (smfr)
Comment 7 2018-11-16 10:10:55 PST
I think this is an animations bug. We don't hit KeyframeEffect::applyPendingAcceleratedActions() when the transition ends.
Simon Fraser (smfr)
Comment 8 2018-11-16 10:20:38 PST
https://trac.webkit.org/changeset/238128/webkit may be related.
Radar WebKit Bug Importer
Comment 9 2018-11-16 10:21:38 PST
<rdar://problem/46130742>
Antoine Quint
Comment 10 2019-12-17 13:32:22 PST
Does not reproduce with ToT for me.
Note You need to log in before you can comment on or make changes to this bug.