Fix documentation on connect time in LoadTimingInfo

net/base/load_timing_info.h

 // 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.
 
 #ifndef NET_BASE_LOAD_TIMING_INFO_H_
 #define NET_BASE_LOAD_TIMING_INFO_H_
 
 #include <stdint.h>
 
 #include "base/time/time.h"
@@ skipping 59 matching lines @@
     // The time spent looking up the host's DNS address.  Null for requests that
     // used proxies to look up the DNS address.  Also null for SOCKS4 proxies,
     // since the DNS address is only looked up after the connection is
     // established, which results in unexpected event ordering.
     // TODO(mmenke):  The SOCKS4 event ordering could be refactored to allow
     //                these times to be non-null.
     base::TimeTicks dns_start;
     base::TimeTicks dns_end;
 
     // The time spent establishing the connection. Connect time includes proxy
-    // connect times (Though not proxy_resolve times), DNS lookup times, time
-    // spent waiting in certain queues, TCP, and SSL time.
+    // connect times (though not proxy_resolve times), time spent waiting in
+    // certain queues, TCP, and SSL time. It does not include DNS lookup time.

[mmenke, 2016/09/08 18:57:34]

Maybe move this up with the "proxy_resolve" times comment?

     // TODO(mmenke):  For proxies, this includes time spent blocking on higher
     //                level socket pools.  Fix this.
     // TODO(mmenke):  Retried connections to the same server should apparently
     //                be included in this time.  Consider supporting that.
     //                Since the network stack has multiple notions of a "retry",
     //                handled at different levels, this may not be worth
     //                worrying about - backup jobs, reused socket failure,
     //                multiple round authentication.
     base::TimeTicks connect_start;
     base::TimeTicks connect_end;
@@ skipping 54 matching lines @@
   // the times that push started and ended. Note that push_end will be null
   // if the request is still being transmitted, i.e. the underlying h2 stream
   // is not closed by the server.
   base::TimeTicks push_start;
   base::TimeTicks push_end;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_LOAD_TIMING_INFO_H_