Closed Bug 1144509 Opened 6 years ago Closed 6 years ago

timeupdate event isn't fired before seeked event


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




Tracking Status
firefox37 --- wontfix
firefox38 --- fixed
firefox39 --- fixed


(Reporter: jya, Assigned: jya)


(Blocks 1 open bug)



(2 files)

Per HTMLMediaElement spec:

" Seeking
16. ⌛ Queue a task to fire a simple event named timeupdate at the element.
17. ⌛ Queue a task to fire a simple event named seeked at the element.

The event timeupdate isn't fired.

This cause failure of the media-source/mediasource-duration.html test
fire timeupdate before seeked event.
Attachment #8579129 - Flags: review?(karlt)
Assignee: nobody → jyavenard
update test. We don't pass on linux and webm as we fail an assert to calculate the duration of the video (6.042 vs 6.045)
Attachment #8579137 - Flags: review?(karlt)
Attachment #8579129 - Flags: review?(karlt) → review+
Attachment #8579137 - Flags: review?(karlt) → review+
Should the test be re-enabled on Mac now?
Perhaps win too?
Flags: needinfo?(jyavenard)
There's more failure with the two following tests, they timeout.

I have found the cause for those (tracked in bug 1143971 and bug 1144617)
Flags: needinfo?(jyavenard)
See Also: → 1144980
Priority: -- → P2
I believe this should be uplifted for 38.
Flags: needinfo?(giles)
Comment on attachment 8579129 [details] [diff] [review]
Part1. Fire timeupdate before seeked event as per spec

Approval Request Comment
[Feature/regressing bug #]: MSE and HTML5 video
[User impact if declined]: Spec compliance, possible stalls after seeking.
[Describe test coverage new/current, TreeHerder]: Landed on m-c, web platform tests.
[Risks and why]: Risk is low. We just queue the missing event, which should have no harmful side-effects.
[String/UUID change made/needed]: None.
Flags: needinfo?(giles)
Attachment #8579129 - Flags: approval-mozilla-aurora?
Blocks: 1144980
Attachment #8579129 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Windows permafail on Aurora:
Flags: needinfo?(jyavenard)
Flags: needinfo?(giles)
You need bug 1142433 uplifted...
Flags: needinfo?(jyavenard)
I believe jya addressed this.
Flags: needinfo?(giles)
You need to log in before you can comment on or make changes to this bug.