From 0d3573976c1742ecfb172512be7a50e972b9a200 Mon Sep 17 00:00:00 2001 From: tommi Date: Thu, 2 Mar 2017 05:59:42 -0800 Subject: [PATCH] Drop out of order frames in VideoRenderFrames. BUG=webrtc:7253 Review-Url: https://codereview.webrtc.org/2724823004 Cr-Commit-Position: refs/heads/master@{#16970} --- webrtc/common_video/video_render_frames.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/webrtc/common_video/video_render_frames.cc b/webrtc/common_video/video_render_frames.cc index 444347d205..31415c5306 100644 --- a/webrtc/common_video/video_render_frames.cc +++ b/webrtc/common_video/video_render_frames.cc @@ -67,8 +67,9 @@ int32_t VideoRenderFrames::AddFrame(VideoFrame&& new_frame) { "%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();