Properly handle partial init segment

RESOLVED INVALID

Status

()

RESOLVED INVALID
4 years ago
4 years ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks: 1 bug)

Trunk
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 obsolete attachments)

(Assignee)

Description

4 years ago
As per W3C spec:
http://w3c.github.io/media-source/#sourcebuffer-segment-parser-loop

We're supposed to handle partial init segment.

Part of bug 1125776, is to be able to fire the update/updateend event once we've completed parsing the init segment.

Support for partial MP4 init segment is almost there as all blocks are there. For WebM it's more complicated.
(Assignee)

Comment 1

4 years ago
will do so in bug 1125776.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
(Assignee)

Comment 2

4 years ago
Created attachment 8557425 [details] [diff] [review]
Add support for partial WebM init segment

Add webm partial init segment support. This is done with a small utility class WebMCheckMetadata that checks if a MediaResource starts with an init segment. This uses libnestegg as their init routine does just that. Ideally we could have expanded WebMBufferedParser but it would have been more complicated and I would have had to learn the inner details of the webm container. Parsing a MediaResource also allows to buffer the data over multiple-calls to appendBuffer with a minimal memory footprint (SourceBufferResource uses ref-counted shared memory block).
Attachment #8557425 - Flags: review?(kinetik)
(Assignee)

Updated

4 years ago
Assignee: nobody → jyavenard
(Assignee)

Comment 3

4 years ago
Created attachment 8557685 [details] [diff] [review]
Add support for partial WebM init segment

Combine the two patches together as the nestegg approach just doesn't work under all circumstances.
Attachment #8557685 - Flags: review?(kinetik)
(Assignee)

Updated

4 years ago
Attachment #8557425 - Attachment is obsolete: true
Attachment #8557425 - Flags: review?(kinetik)
(Assignee)

Comment 4

4 years ago
Comment on attachment 8557685 [details] [diff] [review]
Add support for partial WebM init segment

wrong bug number
Attachment #8557685 - Attachment is obsolete: true
Attachment #8557685 - Flags: review?(kinetik)
You need to log in before you can comment on or make changes to this bug.