Create a separate NetLog source type QUIC_STREAM_FACTORY_JOB

net/log/net_log_event_type_list.h

Index: net/log/net_log_event_type_list.h
diff --git a/net/log/net_log_event_type_list.h b/net/log/net_log_event_type_list.h
index 5e8fe84a73c005098002bf1bea692d8c5e7264d1..4d50905d00d34d1c6e21e7563b822c89aabd049c 100644
--- a/net/log/net_log_event_type_list.h
+++ b/net/log/net_log_event_type_list.h
@@ -1094,6 +1094,15 @@ EVENT_TYPE(HTTP_STREAM_JOB_INIT_CONNECTION)
 //   }
 EVENT_TYPE(HTTP_STREAM_REQUEST_BOUND_TO_JOB)
 
+// Identifies the NetLogSource() for the QuicStreamFactory::Job that the

[eroman, 2017/04/07 01:55:29]

side-note: I wonder why we have parenthesis after NetLogSource() on all of
these? I'll follow up on this, my guess is a mistake that got copy-pasted
everywhere :)

[Zhongyi Shi, 2017/04/07 03:08:53]

Oh, yeah, at least I copied that from HTTP_STREAM_JOB_BOUND_TO_REQUEST.
*facepalm*

+// HttpStreamFactoryImpl::Job was attached to.
+// The event parameters are:
+//  {
+//      "source_dependency": <Source identifier for the QuicStreamFactory::Job
+//                            to which we were attached>,
+//  }
+EVENT_TYPE(HTTP_STREAM_JOB_BOUND_TO_QUIC_STREAM_FACTORY_JOB)
+
 // Identifies the NetLogSource() for the Request that the Job was attached to.
 // The event parameters are:
 //   {
@@ -1632,6 +1641,27 @@ EVENT_TYPE(HTTP2_PROXY_CLIENT_SESSION)
 //   }
 
 // ------------------------------------------------------------------------
+// QuicStreamFactory::Job
+// ------------------------------------------------------------------------
+
+// Measures the time taken to execute the QuicStreamFactory::Job.
+// The event parameters are:
+//   {
+//     "server_id": <The QuicServerId that the Job serves>,
+//   }
+EVENT_TYPE(QUIC_STREAM_FACTORY_JOB)
+
+// Identifies the NetLogSoure() for the HttpStreamFactoryImpl::Job that the

[eroman, 2017/04/07 01:55:29]

typo: NetLogSource

[Zhongyi Shi, 2017/04/07 03:08:53]

Done.

+// Job was attached to.
+// The event parameters are:
+//  {
+//     "source_dependency": <Source identifier for the
+//                           HttpStreamFactoryImpl::Job to which we were
+//                           attached>,
+//  }
+EVENT_TYPE(QUIC_STREAM_FACTORY_JOB_BOUND_TO_HTTP_STREAM_JOB)
+
+// ------------------------------------------------------------------------
 // QuicSession
 // ------------------------------------------------------------------------