Closed
Bug 920867
Opened 11 years ago
Closed 11 years ago
Split audio-specific parts of VideoInfo out into AudioInfo, then wrap Video/AudioInfo in new MediaInfo class
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: kinetik, Assigned: kinetik)
Details
Attachments
(1 file)
45.03 KB,
patch
|
cpearce
:
review+
|
Details | Diff | Splinter Review |
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).
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #810325 -
Flags: review?(cpearce)
Assignee | ||
Comment 2•11 years ago
|
||
(Note that this patch applies on top of bug 905513, but the only difference is the presence of the files in content/media/mediasource.)
Updated•11 years ago
|
Attachment #810325 -
Flags: review?(cpearce) → review+
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/473c72edc9ca
https://hg.mozilla.org/mozilla-central/rev/473c72edc9ca
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in
before you can comment on or make changes to this bug.
Description
•