Closed
Bug 1148234
Opened 9 years ago
Closed 9 years ago
Give MSE subdecoders their own task queue and remove borrowing
Categories
(Core :: Audio/Video: Playback, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: bholley, Unassigned)
References
Details
Attachments
(1 file)
11.91 KB,
patch
|
Details | Diff | Splinter Review |
This would be a win on several levels: * MediaSourceReader would interact with subdecoder readers the same way that the MDSM does, giving us more consistent access patterns to our readers. * We could get rid of the weird scratch task queue that TrackBuffer uses to initialize subdecoders, and just use the one from the subdecoder. * We could tighten up the fuzzy invariants surrounding what it means to "be on the decode task queue". None of these are super important, but certainly nice to have - and I think this should actually be pretty easy to do with the recent promise work. I'm going to take a quick crack at it and see if I get lucky.
Comment 1•9 years ago
|
||
Sub-decoders are being removed shortly.
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to Jean-Yves Avenard [:jya] from comment #1) > Sub-decoders are being removed shortly. Oh? That would certainly change things here. What does the new architecture look like? Is there a bug tracking this?
Flags: needinfo?(jyavenard)
Comment 3•9 years ago
|
||
This is part of bug 1119208. As discussed on IRC ; demuxed samples will now be stored in the TrackBuffers. The MediaSourceReader will only have use for a single decoder/reader per source buffer.
Flags: needinfo?(jyavenard)
Reporter | ||
Comment 4•9 years ago
|
||
Sounds good. I'll upload my WIP just in case and then forget about this.
Depends on: 1119208
Reporter | ||
Comment 5•9 years ago
|
||
Reporter | ||
Updated•9 years ago
|
No longer blocks: MediaMonitor
Updated•9 years ago
|
Component: Audio/Video → Audio/Video: Playback
Comment 6•9 years ago
|
||
we don't have sub decoders with the new MSE.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•