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

Unified Diff: net/quic/core/congestion_control/rtt_stats.cc

Issue 2611613003: Add quic_logging (Closed)
Patch Set: fix failed test? Created 3 years, 11 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: net/quic/core/congestion_control/rtt_stats.cc
diff --git a/net/quic/core/congestion_control/rtt_stats.cc b/net/quic/core/congestion_control/rtt_stats.cc
index da3c14646fc66608d9df89cdd405aa840e3cf654..1fd7a2c95800fb19a6619c1cf52736812ae1ade0 100644
--- a/net/quic/core/congestion_control/rtt_stats.cc
+++ b/net/quic/core/congestion_control/rtt_stats.cc
@@ -6,6 +6,8 @@
#include <cstdlib> // std::abs
+#include "net/quic/platform/api/quic_logging.h"
+
namespace net {
namespace {
@@ -39,9 +41,10 @@ void RttStats::UpdateRtt(QuicTime::Delta send_delta,
QuicTime::Delta ack_delay,
QuicTime now) {
if (send_delta.IsInfinite() || send_delta <= QuicTime::Delta::Zero()) {
- LOG(WARNING) << "Ignoring measured send_delta, because it's is "
- << "either infinite, zero, or negative. send_delta = "
- << send_delta.ToMicroseconds();
+ QUIC_LOG_FIRST_N(WARNING, 3)
+ << "Ignoring measured send_delta, because it's is "
+ << "either infinite, zero, or negative. send_delta = "
+ << send_delta.ToMicroseconds();
return;
}
@@ -73,8 +76,8 @@ void RttStats::UpdateRtt(QuicTime::Delta send_delta,
kOneMinusBeta * mean_deviation_.ToMicroseconds() +
kBeta * std::abs((smoothed_rtt_ - rtt_sample).ToMicroseconds())));
smoothed_rtt_ = kOneMinusAlpha * smoothed_rtt_ + kAlpha * rtt_sample;
- DVLOG(1) << " smoothed_rtt(us):" << smoothed_rtt_.ToMicroseconds()
- << " mean_deviation(us):" << mean_deviation_.ToMicroseconds();
+ QUIC_DVLOG(1) << " smoothed_rtt(us):" << smoothed_rtt_.ToMicroseconds()
+ << " mean_deviation(us):" << mean_deviation_.ToMicroseconds();
}
}
« no previous file with comments | « net/quic/core/congestion_control/prr_sender_test.cc ('k') | net/quic/core/congestion_control/rtt_stats_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698