Wire up rtc_packet_ids to P2PSocketHostTcp, which is required for correct BWE in WebRTC.

content/browser/renderer_host/p2p/socket_host_test_utils.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_P2P_SOCKET_HOST_TEST_UTILS_H_
 #define CONTENT_BROWSER_RENDERER_HOST_P2P_SOCKET_HOST_TEST_UTILS_H_
 
 #include <stdint.h>
 
 #include <string>
@@ skipping 105 matching lines @@
 
 MATCHER_P(MatchIncomingSocketMessage, address, "") {
   if (arg->type() != P2PMsg_OnIncomingTcpConnection::ID)
     return false;
   P2PMsg_OnIncomingTcpConnection::Param params;
   P2PMsg_OnIncomingTcpConnection::Read(
       arg, &params);
   return std::get<1>(params) == address;
 }
 
+MATCHER_P(MatchSendPacketMetrics, rtc_packet_id, "") {
+  if (arg->type() != P2PMsg_OnSendComplete::ID)
+    return false;
+
+  P2PMsg_OnSendComplete::Param params;
+  P2PMsg_OnSendComplete::Read(arg, &params);
+  return std::get<1>(params).rtc_packet_id == rtc_packet_id &&
+         (base::TimeTicks::Now() - std::get<1>(params).send_time)
+                 .InMilliseconds() < 100;

[Taylor_Brandstetter, 2017/04/25 21:34:19]

nit: Could you make the 100 milliseconds an argument of this matcher? That would
make it more clear that it's something the test is testing for.

[Sergey Ulanov, 2017/04/26 00:11:52]

+1. This test may be flaky. Also it currently doesn't verify that send_time is
before now. I suggest passing the time the test was started to this matcher and
the verify here that send_time is between start_time and now.

[Stefan, 2017/04/26 09:02:39]

Done.

+}
+
 #endif  // CONTENT_BROWSER_RENDERER_HOST_P2P_SOCKET_HOST_TEST_UTILS_H_