Bug 1838052 - Allow audio/aacp to always bypass the ORB check r=farre…

…,necko-reviewers,valentin audio/aacp is a variant of acc, since we've allowed acc, accp should be allowed as well. Differential Revision: https://phabricator.services.mozilla.com/D180830
dothq · Jun 13, 2023 · c52e5ee · c52e5ee
1 parent 71e9949
commit c52e5ee
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/netwerk/mime/nsMimeTypes.h b/netwerk/mime/nsMimeTypes.h
@@ -130,6 +130,7 @@
 #define AUDIO_MIDI "audio/x-midi"
 #define AUDIO_MATROSKA "audio/x-matroska"
 #define AUDIO_AAC "audio/aac"
+#define AUDIO_AACP "audio/aacp"
 #define AUDIO_MPEG_TS "audio/mp2t"
 #define AUDIO_MPEG_URL "audio/mpegurl"
 

diff --git a/netwerk/protocol/http/OpaqueResponseUtils.cpp b/netwerk/protocol/http/OpaqueResponseUtils.cpp
@@ -57,7 +57,8 @@ static bool IsOpaqueSafeListedSpecBreakingMIMEType(
   // a better way. Chromium currently allows all "audio/*" and "video/*", but
   // from discussion in bug, we want to try only "audio/mpeg".
   if (aContentType.EqualsLiteral(AUDIO_MP3) ||
-      aContentType.EqualsLiteral(AUDIO_AAC)) {
+      aContentType.EqualsLiteral(AUDIO_AAC) ||
+      aContentType.EqualsLiteral(AUDIO_AACP)) {
     return true;
   }