Cast: Make vp8 3-buffer mode work

chrome/renderer/media/cast_rtp_stream.cc

Index: chrome/renderer/media/cast_rtp_stream.cc
diff --git a/chrome/renderer/media/cast_rtp_stream.cc b/chrome/renderer/media/cast_rtp_stream.cc
index dfcca7ac9c87def7ecdcdd4bf69b579275b43892..657b5659ec17aea928a0f80b1d9bd6fa18f58437 100644
--- a/chrome/renderer/media/cast_rtp_stream.cc
+++ b/chrome/renderer/media/cast_rtp_stream.cc
@@ -208,6 +208,7 @@ bool ToVideoSenderConfig(const CastRtpParams& params,
   if (config->max_frame_rate > 120)
     return false;
   if (params.payload.codec_name == kCodecNameVp8) {
+    config->max_number_of_video_buffers_used = 3;

[Alpha Left Google, 2014/08/25 23:23:44]

Two issues:

1. We do not want to turn on this mode by default. Maybe this should by drive by
extension. For example this can be a codec specific parameter, see here:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/common/exte...

2. This should be a boolean instead of int.

[hubbe, 2014/08/27 04:14:03]

1. Done
2. Agreed, but perhaps in a separate CL.

On 2014/08/25 23:23:44, Alpha wrote:

     config->use_external_encoder = IsHardwareVP8EncodingSupported();
     config->codec = media::cast::CODEC_VIDEO_VP8;
   } else if (params.payload.codec_name == kCodecNameH264) {