Support for audio containers

Support for audio containers

[mangui]

I guess you are referring to audio only playlist with AAC container.
this should come in the coming weeks, i will review the demuxer layer so that it can support multiple containers.
sample streams welcome

[tchakabam]

@mangui How about MP3 support (it can be muxed into MP4). I have a working implementation, does your MP4 transmuxer support this? Otherwise it will be a pleasure to add it :) The thing I got also works with AAC of course.

[mangui]

MP4 transmuxer currently only supports MP4 audio, but adding MP3 should not be an issue, providing that browsers support it : in theory it should be signaled as mp4a.40.34but I didn't get any success on Chrome or FF with MediaSource.isTypeSupported('video/mp4; codecs="mp4a.40.34"');

[mangui]

but if you have a working implementation I would be curious to see how it works :-) and PR welcome

[tchakabam]

So, what I know is

• Chrome supports appending plain MP3 elementary stream to SourceBuffer. It advertises that by MediaSource.isTypeSupported('audio/mpeg')
• Firefox only supports fMP4 ('audio/mp4' will be true I believe). And it supports all the MPEG standard payloads for MP4 audio, which is AAC and MP3. However I am not sure how it signals that exactly :)

So in principle I think I would like to update your MP4 transmuxer (or add one) that will produce MP4 with MPEG audio L3 payload.

In general will be nice to make the whole thing more like a pipeline I think, because we may also want to do audio software decoding and other codecs (like OPUS) or formats (WEBM), and maybe use WebAudio as a backend as well :) What do you think?

[mangui]

AAC support merged upstream

[kylophone]

Thanks for this!

[mangui]

@kylophone, needs some testing, i was able to check against one playlist

Thanks, will do some tests when I get a chance.

[mangui]

for MP3 in MP4, refer to gpac/mp4box.js#78 (comment)

[fdambrosio]

Hi, there's news about mp3 only-audio support?

[mangui]

implemented here https://github.com/dailymotion/hls.js/pull/808
i need some time to review it

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.