Revert of Implement QuicHttpStream::GetLoadTimingInfo

Revert of Implement QuicHttpStream::GetLoadTimingInfo (patchset #7 id:200001 of https://codereview.chromium.org/2324183002/ )

Reason for revert:
URLRequestQuicTest.TestTwoRequests is flaky, see https://crbug.com/647071.

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
> Committed: https://crrev.com/83ac2f22700c563fef8a670ad4c5c57d34032e03
> Cr-Original-Commit-Position: refs/heads/master@{#417969}
> Cr-Commit-Position: refs/heads/master@{#418567}

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/f94df884d60b6b3f345c1575292b557eabd5ee0a
Cr-Commit-Position: refs/heads/master@{#418815}

[grt (UTC plus 2), 2016-09-15 09:43:05]

The CQ bit was checked by grt@chromium.org

[grt (UTC plus 2), 2016-09-15 09:43:06]

Created Revert of Implement QuicHttpStream::GetLoadTimingInfo

[commit-bot: I haz the power, 2016-09-15 09:43:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-15 09:44:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-15 09:46:34]

Description was changed from

==========
Revert of Implement QuicHttpStream::GetLoadTimingInfo (patchset #7 id:200001 of
https://codereview.chromium.org/2324183002/ )

Reason for revert:
URLRequestQuicTest.TestTwoRequests is flaky, see https://crbug.com/647071.

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
> Committed: https://crrev.com/83ac2f22700c563fef8a670ad4c5c57d34032e03
> Cr-Original-Commit-Position: refs/heads/master@{#417969}
> Cr-Commit-Position: refs/heads/master@{#418567}

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
==========

to

==========
Revert of Implement QuicHttpStream::GetLoadTimingInfo (patchset #7 id:200001 of
https://codereview.chromium.org/2324183002/ )

Reason for revert:
URLRequestQuicTest.TestTwoRequests is flaky, see https://crbug.com/647071.

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
> Committed: https://crrev.com/83ac2f22700c563fef8a670ad4c5c57d34032e03
> Cr-Original-Commit-Position: refs/heads/master@{#417969}
> Cr-Commit-Position: refs/heads/master@{#418567}

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/f94df884d60b6b3f345c1575292b557eabd5ee0a
Cr-Commit-Position: refs/heads/master@{#418815}
==========

[commit-bot: I haz the power, 2016-09-15 09:46:34]

Patchset 1 (id:??) landed as
https://crrev.com/f94df884d60b6b3f345c1575292b557eabd5ee0a
Cr-Commit-Position: refs/heads/master@{#418815}