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

Unified Diff: chromecast/common/media/cast_media_client.cc

Issue 2723833002: WebM support for new multipart VP9 string. (Closed)
Patch Set: Use latest ordering (9 parts), Plumb eotf to media client Created 3 years, 10 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
Index: chromecast/common/media/cast_media_client.cc
diff --git a/chromecast/common/media/cast_media_client.cc b/chromecast/common/media/cast_media_client.cc
index 9a4213b898041d1f12c09ce2b25c747e1a16f9cb..cff8d4d1f3fd399bdf339c1c74bae8b5f95765e8 100644
--- a/chromecast/common/media/cast_media_client.cc
+++ b/chromecast/common/media/cast_media_client.cc
@@ -59,7 +59,10 @@ void CastMediaClient::RecordRapporURL(const std::string& metric,
// static
bool CastMediaClient::IsSupportedVideoConfig(::media::VideoCodec codec,
::media::VideoCodecProfile profile,
- int level) {
+ int level,
+ gfx::ColorSpace::TransferID eotf) {
+// TODO(servolk): make use of eotf.
servolk 2017/03/02 18:56:36 Nit: missing closing brace. Also I think it might
chcunningham 2017/03/02 23:13:08 I'm not seeing it. Do you mean the brace for the m
chcunningham 2017/03/07 02:27:56 Done.
+
#if defined(OS_ANDROID)
return supported_profiles_->IsSupportedVideoConfig(
ToCastVideoCodec(codec), ToCastVideoProfile(profile), level);

Powered by Google App Engine
This is Rietveld 408576698