Improve HttpStreamFactory NetLog events

net/http/http_stream_factory_impl.cc

 // 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.
 
 #include "net/http/http_stream_factory_impl.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
@@ skipping 137 matching lines @@
     const HttpRequestInfo& request_info,
     RequestPriority priority,
     const SSLConfig& server_ssl_config,
     const SSLConfig& proxy_ssl_config,
     HttpStreamRequest::Delegate* delegate,
     WebSocketHandshakeStreamBase::CreateHelper*
         websocket_handshake_stream_create_helper,
     HttpStreamRequest::StreamType stream_type,
     const NetLogWithSource& net_log) {
   JobController* job_controller =
-      new JobController(this, delegate, session_, job_factory_.get());
+      new JobController(this, delegate, session_, job_factory_.get(),
+                        /*is_preconnect=*/false, net_log);
   job_controller_set_.insert(base::WrapUnique(job_controller));
 
   Request* request = job_controller->Start(
-      request_info, delegate, websocket_handshake_stream_create_helper, net_log,
+      request_info, delegate, websocket_handshake_stream_create_helper,
       stream_type, priority, server_ssl_config, proxy_ssl_config);
 
   return request;
 }
 
 void HttpStreamFactoryImpl::PreconnectStreams(
     int num_streams,
     const HttpRequestInfo& request_info) {
   SSLConfig server_ssl_config;
   SSLConfig proxy_ssl_config;
   session_->GetSSLConfig(request_info, &server_ssl_config, &proxy_ssl_config);
   // All preconnects should perform EV certificate verification.
   server_ssl_config.verify_ev_cert = true;
   proxy_ssl_config.verify_ev_cert = true;
 
   DCHECK(!for_websockets_);
 
+  NetLogWithSource dummy_netlog_with_source;

[mmenke, 2017/01/25 19:32:17]

Can just inline NetLogWithSource() in the line below.

[xunjieli, 2017/01/25 21:41:48]

Done.

   JobController* job_controller =

[eroman, 2017/01/25 19:55:26]

side-comment: can you update this to be std::unique_ptr<JobController> ?

[xunjieli, 2017/01/25 21:41:48]

Done.

-      new JobController(this, nullptr, session_, job_factory_.get());
+      new JobController(this, nullptr, session_, job_factory_.get(),
+                        /*is_preconnect=*/true, dummy_netlog_with_source);
   job_controller_set_.insert(base::WrapUnique(job_controller));
   job_controller->Preconnect(num_streams, request_info, server_ssl_config,
                              proxy_ssl_config);
 }
 
 const HostMappingRules* HttpStreamFactoryImpl::GetHostMappingRules() const {
   return session_->params().host_mapping_rules;
 }
 
 void HttpStreamFactoryImpl::OnNewSpdySessionReady(
     const base::WeakPtr<SpdySession>& spdy_session,
     bool direct,
     const SSLConfig& used_ssl_config,
     const ProxyInfo& used_proxy_info,
     bool was_alpn_negotiated,
     NextProto negotiated_protocol,
-    bool using_spdy,
-    const NetLogWithSource& net_log) {
+    bool using_spdy) {
   while (true) {
     if (!spdy_session)
       break;
     const SpdySessionKey& spdy_session_key = spdy_session->spdy_session_key();
     // Each iteration may empty out the RequestSet for |spdy_session_key| in
     // |spdy_session_request_map_|. So each time, check for RequestSet and use
     // the first one.
     //
     // TODO(willchan): If it's important, switch RequestSet out for a FIFO
     // queue (Order by priority first, then FIFO within same priority). Unclear
@@ skipping 86 matching lines @@
   DCHECK(!host_port_pair.IsEmpty());
 
   url::SchemeHostPort scheme_host_port("https", host_port_pair.host(),
                                        host_port_pair.port());
 
   return session_->http_server_properties()->SupportsRequestPriority(
       scheme_host_port);
 }
 
 }  // namespace net