Closed Bug 920867 Opened 7 years ago Closed 7 years ago

Split audio-specific parts of VideoInfo out into AudioInfo, then wrap Video/AudioInfo in new MediaInfo class

Categories

(Core :: Audio/Video, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: kinetik, Assigned: kinetik)

Details

Attachments

(1 file)

This makes it clearer which parameters are used for a track, and (in the case of the DASH code and MSE) makes copying all of the video or audio parameters safer (note the bug in the DASH code where it copies mStereoMode as part of the audio params).

Also removes any unnecessary initializations of *Info params that are default initialized ( HasAudio/HasVideo = false).
Attachment #810325 - Flags: review?(cpearce)
(Note that this patch applies on top of bug 905513, but the only difference is the presence of the files in content/media/mediasource.)
Attachment #810325 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/mozilla-central/rev/473c72edc9ca
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.