cast: Support for low-latency interactive mode

media/cast/sender/video_sender.cc

Index: media/cast/sender/video_sender.cc
diff --git a/media/cast/sender/video_sender.cc b/media/cast/sender/video_sender.cc
index ff65423d49ad6ba5523e0fbe792ce636f40cdfb8..02178c8f4a98ca0baaa75cb672224afc24404873 100644
--- a/media/cast/sender/video_sender.cc
+++ b/media/cast/sender/video_sender.cc
@@ -95,6 +95,7 @@ VideoSender::VideoSender(
           video_config.max_frame_rate,
           video_config.min_playout_delay,
           video_config.max_playout_delay,
+          video_config.animated_playout_delay,
           video_config.use_external_encoder
               ? NewFixedCongestionControl(
                     (video_config.min_bitrate + video_config.max_bitrate) / 2)
@@ -105,6 +106,7 @@ VideoSender::VideoSender(
       frames_in_encoder_(0),
       last_bitrate_(0),
       playout_delay_change_cb_(playout_delay_change_cb),
+      low_latency_mode_(false),
       last_reported_deadline_utilization_(-1.0),
       last_reported_lossy_utilization_(-1.0),
       weak_factory_(this) {
@@ -161,6 +163,16 @@ void VideoSender::InsertRawVideoFrame(
       "timestamp", reference_time.ToInternalValue(),
       "rtp_timestamp", rtp_timestamp);
 
+  bool low_latency_mode;
+  if (video_frame->metadata()->GetBoolean(
+          VideoFrameMetadata::INTERACTIVE_CONTENT, &low_latency_mode)) {
+    if (low_latency_mode && !low_latency_mode_) {
+      VLOG(1) << "Interactive mode playout time " << min_playout_delay_;
+      playout_delay_change_cb_.Run(min_playout_delay_);
+    }
+    low_latency_mode_ = low_latency_mode;
+  }
+
   // Drop the frame if either its RTP or reference timestamp is not an increase
   // over the last frame's.  This protects: 1) the duration calculations that
   // assume timestamps are monotonically non-decreasing, and 2) assumptions made
@@ -192,7 +204,18 @@ void VideoSender::InsertRawVideoFrame(
         current_round_trip_time_ * kRoundTripsNeeded +
         base::TimeDelta::FromMilliseconds(kConstantTimeMs),
         max_playout_delay_);
-    if (new_target_delay > target_playout_delay_) {
+    // In case of low latency mode, we prefer frame drops over increasing

[miu, 2015/12/08 00:16:16]

nit: IMO, this comment is more useful in the header file where the
|low_latency_mode_| member is declared.

[Irfan, 2015/12/08 01:30:19]

Also added in header.

+    // playout time.
+    if (!low_latency_mode_ && new_target_delay > target_playout_delay_) {
+      // In case we detect user is no more in an low latency mode and there is

[miu, 2015/12/08 00:16:16]

nit: s/no more/no longer/

[Irfan, 2015/12/08 01:30:18]

Done.

+      // a need to drop a frame, we ensure the playout delay is at-least the
+      // the starting value for playing animated content.
+      // This is intended to minimize freeze when moving from an interactive
+      // session to watching animating content while being limited by end-to-end
+      // delay.
+      VLOG(1) << "Ensure playout time is at least " << animated_playout_delay_;
+      if (new_target_delay < animated_playout_delay_)
+        new_target_delay = animated_playout_delay_;
       VLOG(1) << "New target delay: " << new_target_delay.InMilliseconds();
       playout_delay_change_cb_.Run(new_target_delay);
     }