1293159 - mediasource-play-then-seek-back.html permafailing on Linux OPT

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Jean-Yves Avenard [:jya]]

Spawned from bug 1128069.

It is permafailing on Linux OPT (not debug).

It appears to be related to the "seeked" event handler not being received/handled.

Comment 1

[Jean-Yves Avenard [:jya]]

Attachment: Bug 1293159: Do not use timer when there's only one callback pending.

It is invalid to assume that no other actions can be performed between the time setTimout is called with a timeout value of 0 and the task actually runs.
This causes events to be regularly missed.

Review commit: https://reviewboard.mozilla.org/r/69950/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69950/

Comment 2

[Jean-Yves Avenard [:jya]]

Attachment: Bug 1293159: Update expected results.

The test should pass no, we do not need different expected results file.

Review commit: https://reviewboard.mozilla.org/r/69952/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69952/

Comment 3

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

Comment on attachment 8778784 [details]
Bug 1293159: Update expected results.

https://reviewboard.mozilla.org/r/69952/#review67080

Comment 4

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8778783 [details]
Bug 1293159: Do not use timer when there's only one callback pending.

https://reviewboard.mozilla.org/r/69950/#review67090

Comment 5

[Jean-Yves Avenard [:jya]]

Comment on attachment 8778783 [details]
Bug 1293159: Do not use timer when there's only one callback pending.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69950/diff/1-2/

Comment 6

[Jean-Yves Avenard [:jya]]

Comment on attachment 8778784 [details]
Bug 1293159: Update expected results.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69952/diff/1-2/

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/91fecd006ffa
Do not use timer when there's only one callback pending. r=jwwang
https://hg.mozilla.org/integration/autoland/rev/941503a42e89
Update expected results. r=gerald

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/91fecd006ffa
https://hg.mozilla.org/mozilla-central/rev/941503a42e89