Chromium Code Reviews| Index: content/renderer/media/render_media_client.cc |
| diff --git a/content/renderer/media/render_media_client.cc b/content/renderer/media/render_media_client.cc |
| index ef145b3544d6fc3b0a88f8a83a1e5845f64c2594..9f7e9a8b375064a9c1a8a7a70c60f7e947ce28f6 100644 |
| --- a/content/renderer/media/render_media_client.cc |
| +++ b/content/renderer/media/render_media_client.cc |
| @@ -89,13 +89,37 @@ void RenderMediaClient::RecordRapporURL(const std::string& metric, |
| GetContentClient()->renderer()->RecordRapporURL(metric, url); |
| } |
| -bool RenderMediaClient::IsSupportedVideoConfig(media::VideoCodec codec, |
| - media::VideoCodecProfile profile, |
| - int level) { |
| - switch (codec) { |
| +bool IsHdrColorManagementEnabled() { |
|
hubbe
2017/03/07 19:22:16
Remove?
chcunningham
2017/03/07 23:59:17
Done.
|
| + // TODO(hubbe): Wire this up. |
| + return false; |
| +} |
| + |
| +bool IsTransferFunctionSupported(gfx::ColorSpace::TransferID eotf) { |
| + switch (eotf) { |
| + // Transfers supported without color management. |
| + case gfx::ColorSpace::TransferID::GAMMA22: |
| + case gfx::ColorSpace::TransferID::BT709: |
| + case gfx::ColorSpace::TransferID::SMPTE170M: |
| + case gfx::ColorSpace::TransferID::BT2020_10: |
| + case gfx::ColorSpace::TransferID::BT2020_12: |
| + case gfx::ColorSpace::TransferID::IEC61966_2_1: |
| + return true; |
| + default: |
| + // TODO(hubbe): wire up support for HDR transfers. |
| + return false; |
| + } |
| +} |
| + |
| +bool RenderMediaClient::IsSupportedVideoConfig( |
| + const media::VideoConfig& config) { |
| + // TODO(chcunningham): Query decoders for codec profile support. |
| + switch (config.codec) { |
| + case media::kCodecVP9: |
| + // Color management required for HDR to not look terrible. |
| + return IsTransferFunctionSupported(config.eotf); |
| + |
| case media::kCodecH264: |
| case media::kCodecVP8: |
| - case media::kCodecVP9: |
| case media::kCodecTheora: |
| return true; |