Open Bug 1768562 Opened 3 years ago Updated 3 years ago

Remove duplication in AacCodecSpecificData

Categories

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

Product:
Core
Component:
Audio/Video: Playback
Type:
task

Tracking

()

People

(Reporter: bryce, Unassigned)

References

Details

Following bug 1747760 we'll have an AacCodecSpecificData struct that contains 2 members: mEsDescriptorBinaryBlob and mDecoderConfigDescriptorBinaryBlob. The latter of these is a subset of the former, but for ease of use we're storing both.

While we've duplicated this data historically, rather than doing so it may make more sense to just store the ES Descriptor (the superset), and provide something like a Span to track the decoder config descriptor binary blob within the ES Descriptor.

Our Rust MP4 parser current does the parsing for this data, it may make sense to expose functionality from there to do this. Alternatively, we could add the logic to gecko code, though that risks duplicating the logic.

You need to log in before you can comment on or make changes to this bug.