Fix failed capture handling in WebrtcVideoStream

remoting/protocol/webrtc_frame_scheduler_simple.cc

Index: remoting/protocol/webrtc_frame_scheduler_simple.cc
diff --git a/remoting/protocol/webrtc_frame_scheduler_simple.cc b/remoting/protocol/webrtc_frame_scheduler_simple.cc
index ed037677d65285018d757e6d2e485aaa3499c360..0c44c9ff55df04bb72907ccf060a15cdab582b8b 100644
--- a/remoting/protocol/webrtc_frame_scheduler_simple.cc
+++ b/remoting/protocol/webrtc_frame_scheduler_simple.cc
@@ -178,14 +178,22 @@ bool WebrtcFrameSchedulerSimple::OnFrameCaptured(
 
   base::TimeTicks now = base::TimeTicks::Now();
 
-  if ((!frame || frame->updated_region().is_empty()) && !top_off_is_active_ &&
-      !key_frame_request_) {
-    frame_pending_ = false;
-    ScheduleNextFrame(now);
-    return false;
+  if ((!frame || frame->updated_region().is_empty())) {
+    // If we've failed to capture a frame or captured an empty frame we still
+    // need to encode and send the previous frame when top-off is active or a
+    // key-frame was requested. But it makes sense only when we have a frame to
+    // send, i.e. there is nothing to send if first capture request failed.
+    bool resend_last_frame =
+        captured_first_frame_ && (top_off_is_active_ || key_frame_request_);

[Jamie, 2017/03/21 18:33:36]

Given the name of this variable, I would expect to see the |frame| replaced by
the previous frame here. In fact, this class doesn't even store the previous
frame, which makes me suspect that this isn't the right variable name. Where is
the resend-last-frame logic?

[Sergey Ulanov, 2017/03/21 19:40:26]

The encoder keeps current frame internally (in YUV). For every Encode() call it
updates the content of that buffer from the new frame by copying
updated_region() (while also performing RGB->YUV conversion) and then encodes
that buffer.
So encoder->Encode(nullptr) will return either top-off or key-frame (when the
corresponding flag is set).

+    if (!resend_last_frame) {
+      frame_pending_ = false;
+      ScheduleNextFrame(now);
+      return false;
+    }
   }
 
   if (frame) {
+    captured_first_frame_ = true;
     encoder_bitrate_.SetFrameSize(frame->size());
   }