Closed Bug 1061007 Opened 6 years ago Closed 6 years ago

Handle SourceBuffer appends that are not aligned on a random access point

Categories

(Core :: Audio/Video, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla35

People

(Reporter: kinetik, Assigned: kinetik)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 3 obsolete files)

ContainerParser currently assumes that each append starts on a random access point (e.g. init segment or cluster) and contains a complete chunk.  That's not something that the spec requires so ContainerParser must be extended to handle non-aligned appends.

This is important right now because YouTube use this to append ~64kB chunks as a way of controlling the streaming rate of XmlHttpRequests.
Attached patch wip (obsolete) — Splinter Review
Attachment #8482014 - Attachment is obsolete: true
Attachment #8482184 - Flags: review?(cajbir.bugzilla)
Same as before, but with cruft excluded.
Attachment #8482184 - Attachment is obsolete: true
Attachment #8482184 - Flags: review?(cajbir.bugzilla)
Attachment #8482186 - Flags: review?(cajbir.bugzilla)
Comment on attachment 8482186 [details] [diff] [review]
Handle unaligned appends by retaining WebM parser state across calls.  r=cajbir

Still has cruft.
Attachment #8482186 - Attachment is obsolete: true
Attachment #8482186 - Flags: review?(cajbir.bugzilla)
Attachment #8482188 - Flags: review?(cajbir.bugzilla) → review+
Attachment #8482449 - Flags: review?(cajbir.bugzilla)
(In reply to Matthew Gregan [:kinetik] from comment #6)
> Created attachment 8482449 [details] [diff] [review]
> Only start new decoder on media segment boundary.

What does this patch apply on top of? I get conflicts with SourceBuffer.cpp in ParseStartAndEndTimestamps.
Attachment #8482449 - Flags: review?(cajbir.bugzilla) → review+
Duplicate of this bug: 1062017
https://hg.mozilla.org/mozilla-central/rev/478f4c80061a
https://hg.mozilla.org/mozilla-central/rev/57db771b7b9d
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Blocks: 1041374
Blocks: 1040552
Depends on: 1067785
See Also: → 1124050
You need to log in before you can comment on or make changes to this bug.