Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(271)

Unified Diff: webrtc/modules/video_coding/frame_buffer2.cc

Issue 2980413002: Fix the video buffer size should take rtt into consideration (Closed)
Patch Set: Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/video_coding/frame_buffer2.cc
diff --git a/webrtc/modules/video_coding/frame_buffer2.cc b/webrtc/modules/video_coding/frame_buffer2.cc
index 5be20a2bd5e54576b9fbe797569f4796626343ce..b87c34e70995fc0c1d751071aa5d1daa89de5760 100644
--- a/webrtc/modules/video_coding/frame_buffer2.cc
+++ b/webrtc/modules/video_coding/frame_buffer2.cc
@@ -274,6 +274,9 @@ int FrameBuffer::InsertFrame(std::unique_ptr<FrameObject> frame) {
rtc::CritScope lock(&crit_);
+ if (frame->delayed_by_retransmission())
+ jitter_estimator_->FrameNacked();
philipel2 2017/07/24 14:17:22 I think we should move this inside the if block on
+
int last_continuous_picture_id =
last_continuous_frame_it_ == frames_.end()
? -1

Powered by Google App Engine
This is Rietveld 408576698