1727881 - SVGAnimationElement is missing onbegin/onend/onrepeat event handler IDL attributes

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Philip Jägenstedt [:foolip]]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.159 Safari/537.36

Steps to reproduce:

Tested support using http://mdn-bcd-collector.appspot.com/tests/api/SVGAnimationElement for the purposes of updating the MDN compat data in https://github.com/mdn/browser-compat-data/pull/12113.

Actual results:

The onbegin/onend/onrepeat event handler IDL attributes are not supported, confirmed here:
https://github.com/mozilla/gecko-dev/blob/3e1dc3d1eb4cb2aafd69d9215e8476e61a8e72d9/dom/webidl/SVGAnimationElement.webidl

Note that Firefox does seem to support the events themselves and the content attributes, based on WPT results:
https://wpt.fyi/results/svg/animations?run_id=5714668552192000&run_id=4921134689550336&run_id=6272201545220096&run_id=4808538330234880

Expected results:

Per spec these IDL attributes should be supported:
https://svgwg.org/specs/animations/#InterfaceSVGAnimationElement

They're already supported in Chrome, but not Safari.

Comment 1

[Robert Longson [:longsonr]]

Attachment: Bug 1727881 - Add onbegin/onend/onrepeat event handler IDL attributes to SVGAnimationElement

Comment 2

[Robert Longson [:longsonr]]

https://treeherder.mozilla.org/jobs?repo=try&revision=b4217a7369a768d7f9215c1f10c1995bc5aa9032

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2d63a1325e8
Add onbegin/onend/onrepeat event handler IDL attributes to SVGAnimationElement r=emilio

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/30221 for changes under testing/web-platform/tests

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/b2d63a1325e8

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot