Add logging to the QUIC write path.

net/base/net_log_event_type_list.h

Index: net/base/net_log_event_type_list.h
diff --git a/net/base/net_log_event_type_list.h b/net/base/net_log_event_type_list.h
index 86863f7f1e358c8f3c47506140d76abe581078b2..349c6c72034914dd265f0651ecc2d2dd11fac3fb 100644
--- a/net/base/net_log_event_type_list.h
+++ b/net/base/net_log_event_type_list.h
@@ -1279,6 +1279,14 @@ EVENT_TYPE(QUIC_SESSION_CLOSE_ON_ERROR)
 //   }
 EVENT_TYPE(QUIC_SESSION_PACKET_RECEIVED)
 
+// Session sent a QUIC packet.
+//   {
+//     "encryption_level": <The EncryptionLevel of the packet>
+//     "packet_sequence_number": <The packet's full 64-bit sequence number,
+//                                as a base-10 string.>,
+//     "size": <The size of the packet in bytes>

[eroman, 2013/06/20 22:20:04]

Add a closing brace.

[Ryan Hamilton, 2013/06/20 22:44:16]

Done.

+EVENT_TYPE(QUIC_SESSION_PACKET_SENT)
+
 // Session received a QUIC packet header for a valid packet.
 //   {
 //     "guid": <The 64-bit GUID for this connection, as a base-10 string>,
@@ -1299,6 +1307,15 @@ EVENT_TYPE(QUIC_SESSION_PACKET_HEADER_RECEIVED)
 //   }
 EVENT_TYPE(QUIC_SESSION_STREAM_FRAME_RECEIVED)
 
+// Session sent a STREAM frame.
+//   {
+//     "stream_id": <The id of the stream which this data is for>,
+//     "fin": <True if this is the final data set by the peer on this stream>,
+//     "offset": <Offset in the byte stream where this data starts>,
+//     "length": <Length of the data in this frame>,
+//   }
+EVENT_TYPE(QUIC_SESSION_STREAM_FRAME_SENT)
+
 // Session received an ACK frame.
 //   {
 //     "sent_info": <Details of packet sent by the peer>
@@ -1317,6 +1334,24 @@ EVENT_TYPE(QUIC_SESSION_STREAM_FRAME_RECEIVED)
 //   }
 EVENT_TYPE(QUIC_SESSION_ACK_FRAME_RECEIVED)
 
+// Session sent an ACK frame.
+//   {
+//     "sent_info": <Details of packet sent by the peer>
+//       {
+//         "least_unacked": <Lowest sequence number of a packet sent by the peer
+//                           for which it has not received an ACK>,
+//       }
+//     "received_info": <Details of packet received by the peer>
+//       {
+//         "largest_observed": <The largest sequence number of a packet received
+//                               by (or inferred by) the peer>,
+//         "missing": <List of sequence numbers of packets lower than
+//                     largest_observed which have not been received by the
+//                     peer>,
+//       }
+//   }
+EVENT_TYPE(QUIC_SESSION_ACK_FRAME_SENT)
+
 // Session recevied a RST_STREAM frame.
 //   {
 //     "offset": <Offset in the byte stream which triggered the reset>,
@@ -1325,6 +1360,14 @@ EVENT_TYPE(QUIC_SESSION_ACK_FRAME_RECEIVED)
 //   }
 EVENT_TYPE(QUIC_SESSION_RST_STREAM_FRAME_RECEIVED)
 
+// Session sent a RST_STREAM frame.
+//   {
+//     "offset": <Offset in the byte stream which triggered the reset>,
+//     "error_code": <QuicErrorCode in the frame>,

[eroman, 2013/06/20 22:20:04]

What is this value? If it is a net error code you should call it net_error for
pretty printing. Similarly if it is a quick error you need to call it quic_error

[Ryan Hamilton, 2013/06/20 22:44:16]

Whoops.  Looks like I should have fixed this in the pretty-print CL.  Done.  (It
should have been called quic_rst_stream_error, and the previous as well)

+//     "details": <Human readable description>,
+//   }
+EVENT_TYPE(QUIC_SESSION_RST_STREAM_FRAME_SENT)
+
 // Session received a CONGESTION_FEEDBACK frame.
 //   {
 //     "type": <The specific type of feedback being provided>,
@@ -1346,6 +1389,27 @@ EVENT_TYPE(QUIC_SESSION_RST_STREAM_FRAME_RECEIVED)
 //   }
 EVENT_TYPE(QUIC_SESSION_CONGESTION_FEEDBACK_FRAME_RECEIVED)
 
+// Session received a CONGESTION_FEEDBACK frame.
+//   {
+//     "type": <The specific type of feedback being provided>,
+//     Other per-feedback type details:
+//
+//     for InterArrival:
+//     "accumulated_number_of_lost_packets": <Total number of lost packets
+//                                            over the life of this session>,
+//     "received_packets": <List of strings of the form:
+//                          <sequence_number>@<receive_time_in_ms>>,
+//
+//     for FixRate:
+//     "bitrate_in_bytes_per_second": <The configured bytes per second>,
+//
+//     for TCP:
+//     "accumulated_number_of_lost_packets": <Total number of lost packets
+//                                            over the life of this session>,
+//     "receive_window": <Number of bytes in the receive window>,
+//   }
+EVENT_TYPE(QUIC_SESSION_CONGESTION_FEEDBACK_FRAME_SENT)
+
 // Session received a CONNECTION_CLOSE frame.
 //   {
 //     "error_code": <QuicErrorCode in the frame>,
@@ -1353,6 +1417,13 @@ EVENT_TYPE(QUIC_SESSION_CONGESTION_FEEDBACK_FRAME_RECEIVED)
 //   }
 EVENT_TYPE(QUIC_SESSION_CONNECTION_CLOSE_FRAME_RECEIVED)
 
+// Session received a CONNECTION_CLOSE frame.
+//   {
+//     "error_code": <QuicErrorCode in the frame>,

[eroman, 2013/06/20 22:20:04]

same question

[Ryan Hamilton, 2013/06/20 22:44:16]

Also should have been fixed in earlier CL :<  Should be quic_error.  Fixed. 
Thanks!

+//     "details": <Human readable description>,
+//   }
+EVENT_TYPE(QUIC_SESSION_CONNECTION_CLOSE_FRAME_SENT)
+
 // ------------------------------------------------------------------------
 // QuicHttpStream
 // ------------------------------------------------------------------------