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

Unified Diff: media/base/mime_util_internal.cc

Issue 2803823003: Query LibVPX for VP9 profile support.
Patch Set: 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 | « media/base/decode_capabilities.cc ('k') | media/base/mime_util_unittest.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 9bb409995088b2a0d77ae2bfd2008796b7a5c229..d605cdb835b6b423f1f20f1f7238c0b089e2ecbd 100644
--- a/media/base/mime_util_internal.cc
+++ b/media/base/mime_util_internal.cc
@@ -729,10 +729,19 @@ SupportsType MimeUtil::IsCodecSupported(const std::string& mime_type_lower_case,
ambiguous_platform_support = true;
}
} else if (codec == MimeUtil::VP9 && video_profile != VP9PROFILE_PROFILE0) {
- // We don't know if the underlying platform supports these profiles. Need
- // to add platform level querying to get supported profiles.
- // https://crbug.com/604566
+#if defined(OS_ANDROID)
+ // TODO(chcunningham): Get unambiguous answer for android platform support.
+ // Desktop support is described in calls to media::IsSupportedVideoConfig()
+ // below. In the future Android will use the same path.
+ // https://crbug.com/695264
ambiguous_platform_support = true;
+#else
+ // TODO(chcunningham): Pass Key System configuration through to MediaClient
+ // to describe support for specific profiles. For now we don't know what
+ // Key System is being used, nor what profile's it supports.
+ // https://crbug.com/604566
+ ambiguous_platform_support = is_encrypted;
+#endif
}
AudioCodec audio_codec = MimeUtilToAudioCodec(codec);
« no previous file with comments | « media/base/decode_capabilities.cc ('k') | media/base/mime_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698