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

Unified Diff: content/common/p2p_socket_type.h

Issue 1345583004: Wire up transport sequence number and send time. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments addressed. Created 5 years, 3 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: content/common/p2p_socket_type.h
diff --git a/content/common/p2p_socket_type.h b/content/common/p2p_socket_type.h
index ed77296b28de513dc2fcbacb949afd41199342bc..f38590f3a217ae77f79eb5c2f1fde2eceac8ad74 100644
--- a/content/common/p2p_socket_type.h
+++ b/content/common/p2p_socket_type.h
@@ -10,6 +10,7 @@
#include <string>
+#include "base/time/time.h"
#include "net/base/ip_endpoint.h"
namespace content {
@@ -52,10 +53,17 @@ struct P2PHostAndIPEndPoint {
// Currently, it only carries packet_id but could be expanded to include
// timestamps when packet arrives at various points.
struct P2PSendPacketMetrics {
- P2PSendPacketMetrics() : packet_id(0) {}
- explicit P2PSendPacketMetrics(uint64_t packet_id) : packet_id(packet_id) {}
+ P2PSendPacketMetrics() {}
+ P2PSendPacketMetrics(uint64_t packet_id,
+ int32_t transport_sequence_number,
+ base::TimeTicks send_time_ticks)
+ : packet_id(packet_id),
+ transport_sequence_number(transport_sequence_number),
+ send_time_ticks(send_time_ticks) {}
- uint64_t packet_id;
+ uint64_t packet_id = 0;
+ int32_t transport_sequence_number = -1;
Sergey Ulanov 2015/09/17 17:19:31 It's not clear what transport_sequence_number is a
Stefan 2015/09/18 07:56:51 Done.
+ base::TimeTicks send_time_ticks;
Sergey Ulanov 2015/09/17 17:19:31 nit: suggest calling it |send_time|, i.e. remove _
Stefan 2015/09/18 07:56:51 Done.
};
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698