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

Unified Diff: net/base/mime_util.cc

Issue 160125: Updated media MIME type handling for Chrome and Chromium. (Closed)
Patch Set: Removed mpeg Created 11 years, 5 months 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/base/mime_util.cc
diff --git a/net/base/mime_util.cc b/net/base/mime_util.cc
index 4cbb94dc6b487941eac2feb73f7c3660d74a2577..fb810094548180c673125dd6bc97902093f77e0a 100644
--- a/net/base/mime_util.cc
+++ b/net/base/mime_util.cc
@@ -69,11 +69,9 @@ static const MimeInfo primary_mappings[] = {
{ "image/gif", "gif" },
{ "image/jpeg", "jpeg,jpg" },
{ "image/png", "png" },
-#if defined(GOOGLE_CHROME_BUILD)
{ "video/mp4", "mp4,m4v" },
{ "audio/x-m4a", "m4a" },
{ "audio/mp3", "mp3" },
-#endif
{ "video/ogg", "ogv,ogm" },
{ "audio/ogg", "ogg,oga" },
{ "application/xhtml+xml", "xhtml,xht" }
@@ -180,13 +178,12 @@ static const char* const supported_image_types[] = {
// A list of media types: http://en.wikipedia.org/wiki/Internet_media_type
// A comprehensive mime type list: http://plugindoc.mozdev.org/winmime.php
-
static const char* const supported_media_types[] = {
// Ogg.
"video/ogg",
"audio/ogg",
- // TODO(fbarchard): Remove these for Chromium when layout tests dont use mp4
+#if defined(GOOGLE_CHROME_BUILD)
// MPEG-4.
"video/mp4",
"video/x-m4v",
@@ -196,11 +193,7 @@ static const char* const supported_media_types[] = {
// MP3.
"audio/mp3",
"audio/x-mp3",
-
- // Generic MPEG mime-types.
- // TODO(fbarchard): Remove these when layout tests stop using mpg.
- "audio/mpeg",
- "video/mpeg"
+#endif
};
// Note: does not include javascript types list (see supported_javascript_types)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698