Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7821448df0f4202c66c24715c1edad7b1cdb0c34
      
https://github.com/WebKit/WebKit/commit/7821448df0f4202c66c24715c1edad7b1cdb0c34
  Author: Antoine Quint <[email protected]>
  Date:   2022-11-14 (Mon, 14 Nov 2022)

  Changed paths:
    M 
LayoutTests/imported/w3c/web-platform-tests/web-animations/timing-model/timelines/update-and-send-events-replacement-expected.txt
    M Source/WebCore/style/Styleable.cpp

  Log Message:
  -----------
  [css-transitions] setting transition-property to "none" does not disassociate 
CSS Transition from owning element
https://bugs.webkit.org/show_bug.cgi?id=247884

Reviewed by Antti Koivisto.

We would fail a subtest in 
web-animations/timing-model/timelines/update-and-send-events-replacement.html 
where a CSS Transition
was canceled by setting the transition-property CSS property to "none". We 
would correctly cancel the transition in this situation,
but we would fail to disassociate it from its owning element, the element that 
was the transition's target when it was created.

That association is critical when determining whether an animation, including 
transitions, can be replaced, per
https://drafts.csswg.org/web-animations/#removing-replaced-animations.

We now correctly call DeclarativeAnimation::cancelFromStyle() instead of 
WebAnimation::cancel() when an element's style moves
from having a transition defined for a given property to no longer having one 
in the new style.

* 
LayoutTests/imported/w3c/web-platform-tests/web-animations/timing-model/timelines/update-and-send-events-replacement-expected.txt:
* Source/WebCore/style/Styleable.cpp:
(WebCore::updateCSSTransitionsForStyleableAndProperty):

Canonical link: https://commits.webkit.org/256666@main


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to