Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2213)

Unified Diff: net/base/mime_util.cc

Issue 11416367: Add Opus decode wrapper to media. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased (changed machines) Address most comments. Added TODO for Opus buffering question. Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/base/mime_util.cc
diff --git a/net/base/mime_util.cc b/net/base/mime_util.cc
index f7240f286c8bd85d19190f50195f756da9aa1d6f..b5437ae74f73998237ab9dc82ea9ddea855165b9 100644
--- a/net/base/mime_util.cc
+++ b/net/base/mime_util.cc
@@ -308,6 +308,7 @@ static const char* const common_media_codecs[] = {
"theora",
#endif
"vorbis",
+ "opus",
scherkus (not reviewing) 2012/12/14 21:13:34 this should be added if the flag is enabled perha
"vp8",
"1" // WAVE_FORMAT_PCM.
};
@@ -421,8 +422,8 @@ struct MediaFormatStrict {
};
static const MediaFormatStrict format_codec_mappings[] = {
- { "video/webm", "vorbis,vp8,vp8.0" },
- { "audio/webm", "vorbis" },
+ { "video/webm", "vorbis,opus,vp8,vp8.0" },
scherkus (not reviewing) 2012/12/14 21:13:34 ditto
+ { "audio/webm", "vorbis,opus" },
{ "audio/wav", "1" }
};
@@ -811,6 +812,7 @@ static const char* const kStandardAudioTypes[] = {
"audio/mpeg",
"audio/mpeg3",
"audio/ogg",
+ "audio/opus",
scherkus (not reviewing) 2012/12/14 21:13:34 is this really a mime type? this list is typicall
"audio/vorbis",
"audio/wav",
"audio/webm",

Powered by Google App Engine
This is Rietveld 408576698