Bug 1827684 - Allow audio/aac. r=sefeng,necko-reviewers

It's possible for a streaming server to not start at an ADTS header
sync marker. We need a sliding window sniffer to be able to handle
this, until then we'll include audio/aac in the gradual ORB
transition.

Differential Revision: https://phabricator.services.mozilla.com/D178177
This commit is contained in:
Andreas Farre 2023-05-16 21:40:47 +00:00
Родитель 70ce9a17fd
Коммит b1c0972fc4
1 изменённых файлов: 2 добавлений и 1 удалений

Просмотреть файл

@ -55,7 +55,8 @@ static bool IsOpaqueSafeListedSpecBreakingMIMEType(
// This is not a good solution, but we need this until we solve Bug 1827684 in
// a better way. Chromium currently allows all "audio/*" and "video/*", but
// from discussion in bug, we want to try only "audio/mpeg".
if (StringBeginsWith(aContentType, "audio/mpeg"_ns)) {
if (aContentType.EqualsLiteral(AUDIO_MP3) ||
aContentType.EqualsLiteral(AUDIO_AAC)) {
return true;
}