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

Unified Diff: webrtc/p2p/base/port.cc

Issue 1940493002: Add Stats to Stun ping. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 8 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/p2p/base/port.cc
diff --git a/webrtc/p2p/base/port.cc b/webrtc/p2p/base/port.cc
index 2ee012aca0b7fee9353acbc293787146aad03e8f..e9f0392b81c1a100ab6098aee5c2e77a7173f23a 100644
--- a/webrtc/p2p/base/port.cc
+++ b/webrtc/p2p/base/port.cc
@@ -793,6 +793,8 @@ Connection::Connection(Port* port,
send_rate_tracker_(100u, 10u),
sent_packets_discarded_(0),
sent_packets_total_(0),
+ sent_ping_requests_(0),
+ recv_ping_responses_(0),
reported_(false),
state_(STATE_WAITING),
receiving_timeout_(WEAK_CONNECTION_RECEIVE_TIMEOUT),
@@ -1243,6 +1245,7 @@ void Connection::OnConnectionRequestResponse(ConnectionRequest* request,
}
rtt_ = (RTT_RATIO * rtt_ + rtt) / (RTT_RATIO + 1);
+ recv_ping_responses_++;
MaybeAddPrflxCandidate(request, response);
}
@@ -1291,6 +1294,7 @@ void Connection::OnConnectionRequestSent(ConnectionRequest* request) {
LOG_JV(sev, this) << "Sent STUN ping"
<< ", id=" << rtc::hex_encode(request->id())
<< ", use_candidate=" << use_candidate;
+ sent_ping_requests_++;
}
void Connection::HandleRoleConflictFromPeer() {
@@ -1364,6 +1368,14 @@ size_t Connection::sent_total_packets() {
return sent_packets_total_;
}
+size_t Connection::sent_ping_requests() {
+ return sent_ping_requests_;
+}
+
+size_t Connection::recv_ping_responses() {
+ return recv_ping_responses_;
+}
+
void Connection::MaybeAddPrflxCandidate(ConnectionRequest* request,
StunMessage* response) {
// RFC 5245

Powered by Google App Engine
This is Rietveld 408576698