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

Unified Diff: webrtc/video_engine/vie_channel_group.cc

Issue 1288033008: Update SendTimeHistory to store complete PacketInfo, not just send time (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Rebase Created 5 years, 3 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
« no previous file with comments | « webrtc/modules/remote_bitrate_estimator/test/packet_sender.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/video_engine/vie_channel_group.cc
diff --git a/webrtc/video_engine/vie_channel_group.cc b/webrtc/video_engine/vie_channel_group.cc
index b1a21f09d524a1568fe1623b721a24917dd7c972..3e31bd3933b4da6a6c8f80c41fed7891d1bd5287 100644
--- a/webrtc/video_engine/vie_channel_group.cc
+++ b/webrtc/video_engine/vie_channel_group.cc
@@ -149,7 +149,8 @@ class AdaptedSendTimeHistory : public SendTimeHistory, public SendTimeObserver {
virtual ~AdaptedSendTimeHistory() {}
void OnPacketSent(uint16_t sequence_number, int64_t send_time) override {
- SendTimeHistory::AddAndRemoveOldSendTimes(sequence_number, send_time);
+ PacketInfo info(0, send_time, sequence_number, 0, false);
+ SendTimeHistory::AddAndRemoveOld(info);
}
};
« no previous file with comments | « webrtc/modules/remote_bitrate_estimator/test/packet_sender.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698