Closed
Bug 1127775
Opened 10 years ago
Closed 10 years ago
Properly handle partial init segment
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
INVALID
People
(Reporter: jya, Assigned: jya)
References
(Blocks 1 open bug)
Details
Attachments
(2 obsolete files)
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•10 years ago
|
||
will do so in bug 1125776.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
Assignee | ||
Comment 2•10 years ago
|
||
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•10 years ago
|
Assignee: nobody → jyavenard
Assignee | ||
Comment 3•10 years ago
|
||
Combine the two patches together as the nestegg approach just doesn't work under all circumstances.
Attachment #8557685 -
Flags: review?(kinetik)
Assignee | ||
Updated•10 years ago
|
Attachment #8557425 -
Attachment is obsolete: true
Attachment #8557425 -
Flags: review?(kinetik)
Assignee | ||
Comment 4•10 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.
Description
•