DescriptionRevert of Implement QuicHttpStream::GetLoadTimingInfo (patchset #3 id:120001 of https://codereview.chromium.org/2324183002/ )
Reason for revert:
Causing http://crbug.com/646087
Original issue's description:
> Implement QuicHttpStream::GetLoadTimingInfo
>
> This CL implements QuicHttpStream::GetLoadTimingInfo.
>
> |dns_start| and |dns_end| are obtained in
> QuicStreamFactory::Job::DoResolveHost and
> DoResolveHostComplete. These two values are passed to
> QuicChromiumClientSession's constructor.
>
> |connect_start| and |connect_end| are obtained in
> QuicChromiumClientSession::CrytoConnect() and when
> handshake is confirmed.
>
> |ssl_start| and |ssl_end| are the same as |connect_start|
> and |connect_end| because QUIC always does encryption.
>
> If a session is reused, the connect_timing fields should
> all be null. This CL also sets
> |socket_reused| field of LoadTimingInfo accordingly.
>
> BUG=637051
>
> Committed: https://crrev.com/5608fc1bd4f0a80b6d1996f3f4059f2efa0a0777
> Cr-Commit-Position: refs/heads/master@{#417969}
TBR=rch@chromium.org,xunjieli@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=637051
Committed: https://crrev.com/44c972ebe17064be1d7dbe1c7bdf7c982e56eb02
Cr-Commit-Position: refs/heads/master@{#418045}
Patch Set 1 #Messages
Total messages: 6 (2 generated)
|