New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 604088 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Consider refactoring media track creation in MSE

Project Member Reported by servolk@chromium.org, Apr 16 2016

Issue description

Currently MSE stream parsers create MediaTrack objects before MediaSourceState creates DemuxerStreams, which makes it necessary to maintain a map in MediaTracks collection to eventually allow translating blink track ids into DemuxerStream objects (see MediaTrack::SetDemuxerStreamForMediaTrack and ::OnTrackIdsAssigned). Perhaps we can simplify this by passing an std::vector of MediaTrack objects from stream parsers into MediaSourceState::OnNewConfigs and then set up all the necessary mappings there.
 
Cc: servolk@chromium.org
Components: Internals>Media>Source
Labels: MSEscrubbed Hotlist-CodeHealth
Owner: ----
For a little further context, see also https://codereview.chromium.org/1812543003/#msg36
Status: WontFix (was: Available)
Actually this has been made obsolete by the recently CL https://codereview.chromium.org/1935873002
Now we generate and assign media track ids in demuxers, rather than in blink, and this makes it easier to map track ids to demuxer streams.

Sign in to add a comment