[webkit-changes] [WebKit/WebKit] 6f928c: REGRESSION (262875 at main / iOS 17): fill: 'both' no...

Antoine Quint noreply at github.com
Mon Jul 10 10:33:10 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6f928cc543c3d8287b87fb3c38cf8ca4f5bc2829
      https://github.com/WebKit/WebKit/commit/6f928cc543c3d8287b87fb3c38cf8ca4f5bc2829
  Author: Antoine Quint <graouts at webkit.org>
  Date:   2023-07-10 (Mon, 10 Jul 2023)

  Changed paths:
    A LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/keyframe-effects/effect-value-opacity-replaced-effect-in-shadow-root-expected.html
    A LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/keyframe-effects/effect-value-opacity-replaced-effect-in-shadow-root-ref.html
    A LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/keyframe-effects/effect-value-opacity-replaced-effect-in-shadow-root.html
    M Source/WebCore/animation/KeyframeEffect.cpp
    M Source/WebCore/animation/KeyframeEffectStack.cpp

  Log Message:
  -----------
  REGRESSION (262875 at main / iOS 17): fill: 'both' not respected with animation
https://bugs.webkit.org/show_bug.cgi?id=257861
rdar://111407099

Reviewed by Simon Fraser.

While we fixed a similar issue with 265498 at main, the fix was not complete. Indeed, to correctly
deal with accelerated replaced effects in an effect stack, we must ensure two things:

1. in `KeyframeEffect::applyPendingAcceleratedActionsOrUpdateTimingProperties()`, where we ensure
   the accelerated effect is present on the backing GraphicsLayerCA in the right sort order, we
   not only need to add an `UpdateProperties` accelerated action but also ensure that we mark it
   as running by setting the `m_lastRecordedAcceleratedAction` member.

2. in `KeyframeEffectStack::applyPendingAcceleratedActions()`, where we update all effects in the
   stack to ensure any pending accelerated actions as applied, we must *not* force the running of
   replaced effects if all the accelerated effects in the stack are *not* actively running. In that
   case we must only call `KeyframeEffect::applyPendingAcceleratedActions` to ensure any previously-running
   accelerated effect has a chance to be stopped.

* LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/keyframe-effects/effect-value-opacity-replaced-effect-in-shadow-root-expected.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/keyframe-effects/effect-value-opacity-replaced-effect-in-shadow-root-ref.html: Added.
* LayoutTests/imported/w3c/web-platform-tests/web-animations/animation-model/keyframe-effects/effect-value-opacity-replaced-effect-in-shadow-root.html: Added.
* Source/WebCore/animation/KeyframeEffect.cpp:
(WebCore::KeyframeEffect::applyPendingAcceleratedActionsOrUpdateTimingProperties):
* Source/WebCore/animation/KeyframeEffectStack.cpp:
(WebCore::KeyframeEffectStack::applyPendingAcceleratedActions const):

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




More information about the webkit-changes mailing list