Open Bug 1348212 Opened 3 years ago Updated 3 years ago

Investigate if VideoInfo->mCodecSpecificConfig should be kept

Categories

(Core :: Audio/Video: Playback, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: ayang, Unassigned)

References

Details

This field [1] is added in [2] for b2g only and filled by old b2g demuxer. Since b2g is gone, it should be removed.
However, ogg decoder uses it [3], it's worth to check if this field is required or not.


[1] https://dxr.mozilla.org/mozilla-central/rev/ff04d410e74b69acfab17ef7e73e7397602d5a68/dom/media/MediaInfo.h#322
[2] https://bugzilla.mozilla.org/show_bug.cgi?id=1153876
[3] http://searchfox.org/mozilla-central/source/dom/media/ogg/OggCodecState.cpp#383
Btw, both AudioInfo and VideoInfo have mExtraData and mCodecSpecificConfig. If we plan to keep it, maybe we should move both fields to TrackInfo and have some nice explanations what they are.
See Also: → 1365872
You need to log in before you can comment on or make changes to this bug.