Detect situation when there is no missing send completion signal in P2PSocket implementation.

content/common/p2p_socket_type.h

 // Copyright 2014 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.
 
 // This file defines some basic types used by the P2P-related IPC
 // messages.
 
 #ifndef CONTENT_COMMON_P2P_SOCKET_TYPE_H_
 #define CONTENT_COMMON_P2P_SOCKET_TYPE_H_
 
@@ skipping 30 matching lines @@
   P2PHostAndIPEndPoint() {}
   P2PHostAndIPEndPoint(const std::string& hostname,
                        const net::IPEndPoint& ip_address)
       : hostname(hostname), ip_address(ip_address) {
   }
 
   std::string hostname;
   net::IPEndPoint ip_address;
 };
 
+// Stuct which keeps track of metrics during a send operation on P2P sockets.
+// Currently, it only carries packet_id but could be expanded to include
+// timestamps when packet arrives at various points.

[Sergey Ulanov, 2014/12/16 00:21:43]

Are you actually planning to add other fields in this struct, or are you adding
it just in case? If you don't currently plan to add other fields here then I
don't think you need it. Just pass packet_id.

[guoweis_left_chromium, 2014/12/16 23:12:33]

Yes, one other thing I'd like to track in the future is where time spent.

+struct P2PSendPacketMetrics {
+  P2PSendPacketMetrics() : packet_id(0) {}
+  explicit P2PSendPacketMetrics(uint64 packet_id) : packet_id(packet_id) {}
+
+  uint64 packet_id;
+};
+
 }  // namespace content
 
 #endif  // CONTENT_COMMON_P2P_SOCKET_TYPE_H_