Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a937e1c8a1a3da0a2f63dd3a1f22c59bcea4b5ad
      
https://github.com/WebKit/WebKit/commit/a937e1c8a1a3da0a2f63dd3a1f22c59bcea4b5ad
  Author: Ahmad Saleem <[email protected]>
  Date:   2024-06-19 (Wed, 19 Jun 2024)

  Changed paths:
    M 
LayoutTests/imported/w3c/web-platform-tests/svg/animations/scripted/end-element-on-inactive-element-expected.txt
    M Source/WebCore/svg/SVGAnimationElement.cpp
    M Source/WebCore/svg/animation/SVGSMILElement.h

  Log Message:
  -----------
  Ignore instance times from endElement*() for inactive timed elements

https://bugs.webkit.org/show_bug.cgi?id=200259

Reviewed by Said Abou-Hallawa.

This patch aligns WebKit with Gecko / Firefox and Blink / Chromium.

Inspired by: 
https://github.com/chromium/chromium/commit/810144f1502198bb66cd265a2ce9a16ccd106dd2

When endElementAt()/endElement() is trying to add a new instance time
and there's no active interval, just ignore the new instance time.

Spec [1]:

   "While the element is not active, any end specification of the
   event is ignored."

[1] 
https://www.w3.org/TR/2001/REC-smil-animation-20010904/#Timing-EventSensitivity

* Source/WebCore/svg/SVGAnimationElement.cpp:
(WebCore::SVGAnimationElement::endElementAt):
* Source/WebCore/svg/animation/SVGSMILElement.h:
(WebCore::SVGSMILElement::activeState const):
* 
LayoutTests/imported/w3c/web-platform-tests/svg/animations/scripted/end-element-on-inactive-element-expected.txt:
 Rebaselined

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to