[webkit-changes] [WebKit/WebKit] 53782b: [scroll-animations] ready task isn't scheduled whe...

Antoine Quint noreply at github.com
Sun Oct 20 22:04:02 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 53782b6c3b151cebfa41fd40e1c2edc2b23ef7c6
      https://github.com/WebKit/WebKit/commit/53782b6c3b151cebfa41fd40e1c2edc2b23ef7c6
  Author: Antoine Quint <graouts at webkit.org>
  Date:   2024-10-20 (Sun, 20 Oct 2024)

  Changed paths:
    M LayoutTests/imported/w3c/web-platform-tests/scroll-animations/scroll-timelines/setting-timeline.tentative-expected.txt
    M Source/WebCore/animation/WebAnimation.cpp

  Log Message:
  -----------
  [scroll-animations] ready task isn't scheduled when switching between monotonic and progress-based timelines
https://bugs.webkit.org/show_bug.cgi?id=281838
rdar://138282058

Reviewed by Simon Fraser.

We made an ill-advised optimization when implementing the revised "setting the timeline
of an animation" procedure from Web Animations Level 2. Indeed, we must compute the
"previous play state" prior to updating the timeline member since doing so may change
that value. We now follow the spec more closely and this addresses some WPT test failures.

* LayoutTests/imported/w3c/web-platform-tests/scroll-animations/scroll-timelines/setting-timeline.tentative-expected.txt:
* Source/WebCore/animation/WebAnimation.cpp:
(WebCore::WebAnimation::setTimeline):

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list