haikuwebkit/LayoutTests/webanimations/transition-restart-after-st...


PASS Forcing a style update in a transitionend event for an accelerated animation should not prevent another transition for the same property from running later.
REGRESSION (r263729): transform transition doesn't restart https://bugs.webkit.org/show_bug.cgi?id=218011 Source/WebCore: Reviewed by Dean Jackson. <rdar://problem/70035130> In the case of accelerated animations, style for the targeted element is not updated while the animation is in flight since the animation is performed by Core Animation. This means that by the time the "transitionend" event is fired, a transition has not yet performed style updates for the element that would have already been performed for a software animations and thus the updates to the completed and running transition maps performed under AnimationTimeline::updateCSSTransitionsForStyleableAndProperty() may be out of sync with the assumptions made in DocumentTimelinesController::updateAnimationsAndSendEvents() that a transition newly marked as finished should be added to the list of completed transitions. Indeed, in the newly-added test, if a style update is forced during the "transitionend" event listener before also clearing the transition styles, two style updates, and thus two calls to updateCSSTransitionsForStyleableAndProperty(), will have been performed by the time updateAnimationsAndSendEvents() runs again and collects completed transitions, during which time the transition in question will have been removed from the list of running transitions and added to the list of completed transitions (first update) and subsequently removed from the list of completed transitions (second update). At this point, setting styles that would start a new transition for this property will not yield a transition since we won't be able to satisfy the requirement that the new target value does not match that of the completed transition. In this change we stop assuming that just because updateAnimationsAndSendEvents() sees a transition as finished for the first time it implies that the transition is still considered a running transition. As such we only add a transition to the list of completed transitions should it be in the list of running transitions. Test: webanimations/transition-restart-after-style-recalc-during-transitionend.html * animation/DocumentTimeline.cpp: (WebCore::DocumentTimeline::transitionDidComplete): LayoutTests: <rdar://problem/70035130> Reviewed by Dean Jackson. Add a new test that checks that forcing a style update during dispatch of the "transitionend" event does not prevent the completed transition from starting again if reset. * webanimations/transition-restart-after-style-recalc-during-transitionend-expected.txt: Added. * webanimations/transition-restart-after-style-recalc-during-transitionend.html: Added. Canonical link: https://commits.webkit.org/230749@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@268809 268f45cc-cd09-0410-ab3c-d52691b4dbfc 2020-10-21 18:10:52 +00:00
			`PASS Forcing a style update in a transitionend event for an accelerated animation should not prevent another transition for the same property from running later.`