Drop out of order frames in VideoRenderFrames.

webrtc/common_video/video_render_frames.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 49 matching lines @@
                  "%s: frame too long into the future, timestamp=%u.",
                  __FUNCTION__, new_frame.timestamp());
     return -1;
   }
 
   if (new_frame.render_time_ms() < last_render_time_ms_) {
     WEBRTC_TRACE(kTraceWarning, kTraceVideoRenderer, -1,
                  "%s: frame scheduled out of order, render_time=%u, latest=%u.",
                  __FUNCTION__, new_frame.render_time_ms(),
                  last_render_time_ms_);
-    // TODO(mflodman): Decide what to do when this happens.
-    // See bug: https://bugs.chromium.org/p/webrtc/issues/detail?id=7253
+    // For more details, see bug:
+    // https://bugs.chromium.org/p/webrtc/issues/detail?id=7253
+    return -1;
   }
 
   last_render_time_ms_ = new_frame.render_time_ms();
   incoming_frames_.emplace_back(std::move(new_frame));
 
   if (incoming_frames_.size() > kMaxIncomingFramesBeforeLogged)
     LOG(LS_WARNING) << "Stored incoming frames: " << incoming_frames_.size();
   return static_cast<int32_t>(incoming_frames_.size());
 }
 
@@ skipping 16 matching lines @@
                                   render_delay_ms_ -
                                   rtc::TimeMillis();
   return time_to_release < 0 ? 0u : static_cast<uint32_t>(time_to_release);
 }
 
 bool VideoRenderFrames::HasPendingFrames() const {
   return !incoming_frames_.empty();
 }
 
 }  // namespace webrtc