1642157 - 2 testing/web-platform/meta/svg/animations/seeking-events tests are TIMEOUT

Status: RESOLVED FIXED

(Core :: SVG, defect, P3)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

I find these two tests are always timing out:
/svg/animations/seeking-events-7.html
/svg/animations/seeking-events-8.html

I verified this locally on my windows 10 machine, no subtests pass just timeout. There are no errors in the console.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

:svoisen, I see that you are the triage owner for this component- could you help me find the right person to determine if these tests are supported, tests need changes, or a bug in the browser?

Comment 2

[Boris Chiou [:boris]]

This may be a bug in Firefox. It seems we don't dispatch beginEvent and endEvent while seeking forward/backward. This is introduced by this Chrome patch: https://chromium-review.googlesource.com/c/chromium/src/+/2020947.

Comment 3

[Brian Birtles (:birtles)]

(In reply to Boris Chiou [:boris] from comment #2)

This may be a bug in Firefox. It seems we don't dispatch beginEvent and endEvent while seeking forward/backward. This is introduced by this Chrome patch: https://chromium-review.googlesource.com/c/chromium/src/+/2020947.

I haven't looked at this particular test case, but in general I think that's deliberate and was something we were going to spec. If you seek a long way, there could be thousands of begin/end events that could be dispatched but doing so makes seeking too expensive so we should only dispatch the minimum number to represent the change in state. That's another reason why we've been recommending authors use syncbase timing instead of event-based timing.

For what it's worth, that was the motivation behind only dispatching one animationiteration event per frame in https://drafts.csswg.org/css-animations-2/#event-dispatch .

So we I think we should push back on that test.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

:birtles, is the recommendation to fix the test?

Comment 5

[Brian Birtles (:birtles)]

(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #4)

:birtles, is the recommendation to fix the test?

We probably can't without standards discussion. I think skipping it would be fine until we do that, however.

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: Bug 1642157 - mark 2 svg animation tests as backlog (tier-2) due to not supported. r=birtles

mark 2 svg animation tests as backlog (tier-2) due to not supported

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c0fed15df545
mark 2 svg animation tests as backlog (tier-2) due to not supported. r=birtles

Comment 8

[Cristina Coroiu [:ccoroiu]]

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