1148234 - Give MSE subdecoders their own task queue and remove borrowing

Status: RESOLVED WONTFIX

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

Description

[Bobby Holley (:bholley)]

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

[Jean-Yves Avenard [:jya]]

Sub-decoders are being removed shortly.

Comment 2

[Bobby Holley (:bholley)]

(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?

Comment 3

[Jean-Yves Avenard [:jya]]

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.

Comment 4

[Bobby Holley (:bholley)]

Sounds good. I'll upload my WIP just in case and then forget about this.

Comment 5

[Bobby Holley (:bholley)]

Attachment: WIP

Comment 6

[Jean-Yves Avenard [:jya]]

we don't have sub decoders with the new MSE.