Closed Bug 1119119 Opened 6 years ago Closed 6 years ago

MSE: calling appendBuffer with no data doesn't emit updateend


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




Tracking Status
firefox36 --- fixed
firefox37 --- fixed
firefox38 --- fixed


(Reporter: jya, Assigned: jya)


(Blocks 1 open bug)



(2 files, 2 obsolete files)

As per MSE spec:

We're supposed to run the segment parser loop algorithm.
Which states:

1.Loop Top: If the input buffer is empty, then jump to the need more data step below.

7.Need more data: Return control to the calling algorithm.

Which upon return:
3.Set the updating attribute to false.
4.Queue a task to fire a simple event named update at this SourceBuffer object.
5.Queue a task to fire a simple event named updateend at this SourceBuffer object.

Right now we abort, and as such no update and updateend events are fired.

This is what cause errors in testing/web-platform/tests/media-source/interfaces.html (and if we apply bug 1118589 now causes timeouts as no error is immediately returned)
Attached patch Update web platform tests (obsolete) — Splinter Review
Update webref test results
Attachment #8546286 - Flags: review?(karlt)
Assignee: nobody → jyavenard
Attachment #8546286 - Flags: review?(karlt) → review+
Do not error if data appended is empty, and properly issue update/updateend event
Attachment #8546301 - Flags: review?(cajbir.bugzilla)
Attachment #8546301 - Flags: review?(cajbir.bugzilla) → review+
Priority: -- → P2
Rebase, so it doesn't rely on async appendBuffer
Attachment #8546301 - Attachment is obsolete: true
Use ./mach web-platform-tests-update to update patch instead. Yield exactly what I had done manually with extra CR
Attachment #8546286 - Attachment is obsolete: true
Comment on attachment 8550027 [details] [diff] [review]
Do not abort when calling appendBuffer with no data

Approval Request Comment
[Feature/regressing bug #]: MSE
[User impact if declined]: Less consistent testing, possible problems with Youtube playback.
[Describe test coverage new/current, TBPL]: stable on inbound, presuming green on m-c.
[Risks and why]: MSE-specific change. Main risk is web platform test result variance from un-uplifted patches.
[String/UUID change made/needed]: None.

This request applies to all patches in this bug.
Attachment #8550027 - Flags: approval-mozilla-beta?
Attachment #8550027 - Flags: approval-mozilla-aurora?
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Attachment #8550027 - Flags: approval-mozilla-beta?
Attachment #8550027 - Flags: approval-mozilla-beta+
Attachment #8550027 - Flags: approval-mozilla-aurora?
Attachment #8550027 - Flags: approval-mozilla-aurora+
This doesn't apply cleanly to beta. I'll need to do a backport before we can uplift this batch.
Or rather, figure out what the difference is, since this code should be the same on both branches. Looks like something with the bug 1118123 backout on 37.
you likely would need bug 1118123 yes...
this bug is non-important, it's only required to make bug 1120084 land.
Ok, I traced the history. Bug 1118123 landed on firefox 37 when it was m-c, on January 9. I uplifted to 36 (then aurora) but backed it out per Jean-Yves' request. On January 12, m-c became firefox 38 and all the changed on 37 were merged into aurora. That explains why the patches from this bug apply to aurora 37, but not beta 36. I was confused because I remembered backing them out of Aurora, forgetting that was before branch uplift.
Flags: qe-verify-
This has automatic coverage in the web-ref test: media-source/mediasource-append-buffer.html

this test that appendBuffer with no data properly emits update/updateend
You need to log in before you can comment on or make changes to this bug.