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

Unified Diff: media/base/mime_util_internal.cc

Issue 2805553004: Wire up MediaCapabilities is_supported to MimeUtil (Closed)
Patch Set: Remove test for theora - not supported on android Created 3 years, 8 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 | « content/test/BUILD.gn ('k') | media/blink/webmediacapabilitiesclient_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/mime_util_internal.cc
diff --git a/media/base/mime_util_internal.cc b/media/base/mime_util_internal.cc
index affb8cc36aeb6869efed4cf3b17f03f8c646f309..9b9471c4347345a07fdb83ca527e027794c416b6 100644
--- a/media/base/mime_util_internal.cc
+++ b/media/base/mime_util_internal.cc
@@ -415,8 +415,10 @@ SupportsType MimeUtil::IsSupportedMediaFormat(
const std::string mime_type_lower_case = base::ToLowerASCII(mime_type);
MediaFormatMappings::const_iterator it_media_format_map =
media_format_map_.find(mime_type_lower_case);
- if (it_media_format_map == media_format_map_.end())
+ if (it_media_format_map == media_format_map_.end()) {
+ DVLOG(3) << __func__ << " Unrecognized mime type: " << mime_type;
return IsNotSupported;
+ }
if (it_media_format_map->second.empty()) {
// We get here if the mimetype does not expect a codecs parameter.
« no previous file with comments | « content/test/BUILD.gn ('k') | media/blink/webmediacapabilitiesclient_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698