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

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_sender.cc

Issue 1213603002: Remove ResetStatistics from RTP feedback. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Remove more data counter resetting. Created 5 years, 6 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/rtp_rtcp/source/rtp_sender.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
index 89231351697665320a2c0c505861cc55b0d202e1..11c486127d10e495f69432d5655b2e01f92d043d 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
@@ -1101,26 +1101,6 @@ uint16_t RTPSender::AllocateSequenceNumber(uint16_t packets_to_send) {
return first_allocated_sequence_number;
}
-void RTPSender::ResetDataCounters() {
- uint32_t ssrc;
- uint32_t ssrc_rtx;
- bool report_rtx;
- {
- CriticalSectionScoped ssrc_lock(send_critsect_.get());
- ssrc = ssrc_;
- ssrc_rtx = ssrc_rtx_;
- report_rtx = rtx_ != kRtxOff;
- }
- CriticalSectionScoped lock(statistics_crit_.get());
- rtp_stats_ = StreamDataCounters();
- rtx_rtp_stats_ = StreamDataCounters();
- if (rtp_stats_callback_) {
- rtp_stats_callback_->DataCountersUpdated(rtp_stats_, ssrc);
- if (report_rtx)
- rtp_stats_callback_->DataCountersUpdated(rtx_rtp_stats_, ssrc_rtx);
- }
-}
-
void RTPSender::GetDataCounters(StreamDataCounters* rtp_stats,
StreamDataCounters* rtx_stats) const {
CriticalSectionScoped lock(statistics_crit_.get());

Powered by Google App Engine
This is Rietveld 408576698