Commit 3e588a3a authored by mgersh's avatar mgersh Committed by Commit bot

Fix documentation on connect time in LoadTimingInfo

The comment on connect_start and connect_end incorrectly stated that DNS
lookup time is included in connect time. The comment above about order
of events has the correct order.

Review-Url: https://codereview.chromium.org/2326693002
Cr-Commit-Position: refs/heads/master@{#417385}
parent 82d42c55
......@@ -77,8 +77,8 @@ struct NET_EXPORT LoadTimingInfo {
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 or DNS lookup times), time spent
// waiting in certain queues, TCP, and SSL time.
// 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
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment