Commit 2da6f193 authored by jchuang's avatar jchuang Committed by Commit bot

Cast: Prefer VP8 over H.264 for hardware encoder

BUG=469409
TEST=Test on Nyan, which has both H.264 and VP8 HW encoder.
     It should choose VP8 instead of H.264.

Review URL: https://codereview.chromium.org/1034433002

Cr-Commit-Position: refs/heads/master@{#324374}
parent 278f5c29
...@@ -151,9 +151,18 @@ std::vector<CastRtpParams> SupportedAudioParams() { ...@@ -151,9 +151,18 @@ std::vector<CastRtpParams> SupportedAudioParams() {
std::vector<CastRtpParams> SupportedVideoParams() { std::vector<CastRtpParams> SupportedVideoParams() {
std::vector<CastRtpParams> supported_params; std::vector<CastRtpParams> supported_params;
// Prefer VP8 over H.264 for hardware encoder.
if (IsHardwareVP8EncodingSupported())
supported_params.push_back(CastRtpParams(DefaultVp8Payload()));
if (IsHardwareH264EncodingSupported()) if (IsHardwareH264EncodingSupported())
supported_params.push_back(CastRtpParams(DefaultH264Payload())); supported_params.push_back(CastRtpParams(DefaultH264Payload()));
supported_params.push_back(CastRtpParams(DefaultVp8Payload()));
// Propose the default software VP8 encoder, if no hardware encoders are
// available.
if (supported_params.empty())
supported_params.push_back(CastRtpParams(DefaultVp8Payload()));
return supported_params; return supported_params;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment