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

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

Issue 2980413002: Fix the video buffer size should take rtt into consideration (Closed)
Patch Set: MODIFY address small redesign issues from the comments 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.h
diff --git a/webrtc/modules/video_coding/frame_buffer2.h b/webrtc/modules/video_coding/frame_buffer2.h
index 307bb85b46c339d7121d0f5ea4187a9f7897d727..4584ef0ee56268126053fd24dc52a26dc545f1d8 100644
--- a/webrtc/modules/video_coding/frame_buffer2.h
+++ b/webrtc/modules/video_coding/frame_buffer2.h
@@ -73,6 +73,12 @@ class FrameBuffer {
// return immediately.
void Stop();
+ // Updates the RTT for jitter buffer estimation.
philipel 2017/07/28 16:11:44 Just comment with "Updates the RTT for jitter buff
+ //
+ // Input:
+ // - rttMs : RTT in ms
+ void UpdateRtt(int64_t rttMs);
philipel 2017/07/28 16:11:44 rttMs --> rtt_ms
+
private:
struct FrameKey {
FrameKey() : picture_id(0), spatial_layer(0) {}

Powered by Google App Engine
This is Rietveld 408576698