Commit 7f7ef67c authored by jiajia.qin's avatar jiajia.qin Committed by Commit bot

Add 'audio/aac' mimetype support

In old implementation, 'audio/aac' is not supported when using
IsSupportedMediaMimeType to check. That will block aac audio to play.

Review URL: https://codereview.chromium.org/895523002

Cr-Commit-Position: refs/heads/master@{#314952}
parent 8e3e03ec
...@@ -359,6 +359,7 @@ static const char* const proprietary_media_types[] = { ...@@ -359,6 +359,7 @@ static const char* const proprietary_media_types[] = {
"audio/mp3", "audio/mp3",
"audio/x-mp3", "audio/x-mp3",
"audio/mpeg", "audio/mpeg",
"audio/aac",
#if defined(ENABLE_MPEG2TS_STREAM_PARSER) #if defined(ENABLE_MPEG2TS_STREAM_PARSER)
// MPEG-2 TS. // MPEG-2 TS.
......
...@@ -231,6 +231,7 @@ TEST(MimeUtilTest, CommonMediaMimeType) { ...@@ -231,6 +231,7 @@ TEST(MimeUtilTest, CommonMediaMimeType) {
EXPECT_TRUE(IsSupportedMediaMimeType("audio/mp3")); EXPECT_TRUE(IsSupportedMediaMimeType("audio/mp3"));
EXPECT_TRUE(IsSupportedMediaMimeType("audio/x-mp3")); EXPECT_TRUE(IsSupportedMediaMimeType("audio/x-mp3"));
EXPECT_TRUE(IsSupportedMediaMimeType("audio/mpeg")); EXPECT_TRUE(IsSupportedMediaMimeType("audio/mpeg"));
EXPECT_TRUE(IsSupportedMediaMimeType("audio/aac"));
#if defined(ENABLE_MPEG2TS_STREAM_PARSER) #if defined(ENABLE_MPEG2TS_STREAM_PARSER)
EXPECT_TRUE(IsSupportedMediaMimeType("video/mp2t")); EXPECT_TRUE(IsSupportedMediaMimeType("video/mp2t"));
...@@ -246,6 +247,7 @@ TEST(MimeUtilTest, CommonMediaMimeType) { ...@@ -246,6 +247,7 @@ TEST(MimeUtilTest, CommonMediaMimeType) {
EXPECT_FALSE(IsSupportedMediaMimeType("audio/mp3")); EXPECT_FALSE(IsSupportedMediaMimeType("audio/mp3"));
EXPECT_FALSE(IsSupportedMediaMimeType("audio/x-mp3")); EXPECT_FALSE(IsSupportedMediaMimeType("audio/x-mp3"));
EXPECT_FALSE(IsSupportedMediaMimeType("audio/mpeg")); EXPECT_FALSE(IsSupportedMediaMimeType("audio/mpeg"));
EXPECT_FALSE(IsSupportedMediaMimeType("audio/aac"));
#endif // USE_PROPRIETARY_CODECS #endif // USE_PROPRIETARY_CODECS
EXPECT_FALSE(IsSupportedMediaMimeType("video/mp3")); EXPECT_FALSE(IsSupportedMediaMimeType("video/mp3"));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment