RTCIceCandidatePairStats.[total/current]RoundTripTime collected.

webrtc/p2p/base/port_unittest.cc

 /*
  *  Copyright 2004 The WebRTC Project Authors. All rights reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 1770 matching lines @@
   rconn->Ping(0);
   EXPECT_EQ(5U, rconn->stats().sent_ping_requests_total);
   EXPECT_EQ(3U, rconn->stats().sent_ping_requests_before_first_response);
 
   // Response should include same ping count.
   retransmit_attr = msg->GetUInt32(STUN_ATTR_RETRANSMIT_COUNT);
   ASSERT_TRUE(retransmit_attr != NULL);
   EXPECT_EQ(2U, retransmit_attr->value());
 }
 
+TEST_F(PortTest, TestRoundTripTime) {
+  std::unique_ptr<TestPort> lport(
+      CreateTestPort(kLocalAddr1, "lfrag", "lpass"));
+  std::unique_ptr<TestPort> rport(
+      CreateTestPort(kLocalAddr2, "rfrag", "rpass"));
+  lport->SetIceRole(cricket::ICEROLE_CONTROLLING);
+  lport->SetIceTiebreaker(kTiebreaker1);
+  rport->SetIceRole(cricket::ICEROLE_CONTROLLED);
+  rport->SetIceTiebreaker(kTiebreaker2);
+
+  lport->PrepareAddress();
+  rport->PrepareAddress();
+  ASSERT_FALSE(rport->Candidates().empty());
+  Connection* lconn = lport->CreateConnection(
+      rport->Candidates()[0], Port::ORIGIN_MESSAGE);
+
+  EXPECT_FALSE(lconn->stats().total_round_trip_time_ms);
+  EXPECT_FALSE(lconn->stats().current_round_trip_time_ms);
+
+  lconn->ReceivedPingResponse(10, "id");

[Taylor Brandstetter, 2017/02/25 01:31:29]

ReceivedPingResponse is something that *should* be private... It would be
preferable to test an actual binding request/response with delay, simulating the
delay with set_delay_mean and UpdateDelayDistribution on the
VirtualSocketServer, and simulating time with a ScopedFakeClock as used in some
other tests.

[pthatcher1, 2017/02/27 18:07:21]

I agree.  TestHandleStunMessage is an example of how to do build responses, and
TestHandleStunBindingIndication an example of how to use SignalReadPacket.

[hbos, 2017/02/27 19:51:34]

I updated the test to use Ping much like the real scenario and ScopedFakeClock
to pass time. Passing messages between lconn and rconn using TestPort's
last_stun_buf/OnReadPacket without having to touch ReceivedPingResponse.

+  ASSERT_TRUE(lconn->stats().total_round_trip_time_ms);
+  ASSERT_TRUE(lconn->stats().current_round_trip_time_ms);
+  EXPECT_EQ(*lconn->stats().total_round_trip_time_ms, 10u);
+  EXPECT_EQ(*lconn->stats().current_round_trip_time_ms, 10u);
+
+  lconn->ReceivedPingResponse(20, "id");
+  ASSERT_TRUE(lconn->stats().total_round_trip_time_ms);
+  ASSERT_TRUE(lconn->stats().current_round_trip_time_ms);
+  EXPECT_EQ(*lconn->stats().total_round_trip_time_ms, 30u);
+  EXPECT_EQ(*lconn->stats().current_round_trip_time_ms, 20u);
+
+  lconn->ReceivedPingResponse(30, "id");
+  ASSERT_TRUE(lconn->stats().total_round_trip_time_ms);
+  ASSERT_TRUE(lconn->stats().current_round_trip_time_ms);
+  EXPECT_EQ(*lconn->stats().total_round_trip_time_ms, 60u);
+  EXPECT_EQ(*lconn->stats().current_round_trip_time_ms, 30u);
+}
+
 TEST_F(PortTest, TestUseCandidateAttribute) {
   std::unique_ptr<TestPort> lport(
       CreateTestPort(kLocalAddr1, "lfrag", "lpass"));
   std::unique_ptr<TestPort> rport(
       CreateTestPort(kLocalAddr2, "rfrag", "rpass"));
   lport->SetIceRole(cricket::ICEROLE_CONTROLLING);
   lport->SetIceTiebreaker(kTiebreaker1);
   rport->SetIceRole(cricket::ICEROLE_CONTROLLED);
   rport->SetIceTiebreaker(kTiebreaker2);
 
@@ skipping 961 matching lines @@
       port->CreateConnection(candidate, Port::ORIGIN_MESSAGE);
   EXPECT_NE(conn1, conn2);
   conn_in_use = port->GetConnection(address);
   EXPECT_EQ(conn2, conn_in_use);
   EXPECT_EQ(2u, conn_in_use->remote_candidate().generation());
 
   // Make sure the new connection was not deleted.
   rtc::Thread::Current()->ProcessMessages(300);
   EXPECT_TRUE(port->GetConnection(address) != nullptr);
 }