Revert of QUIC - track the time it takes for OnStreamReadyCallback() to

Revert of QUIC - track the time it takes for OnStreamReadyCallback() to
be called. (patchset #1 of https://codereview.chromium.org/1618443002/ )

Reason for revert:
We have collected the data.

Manaually reverted the following CL that collected data on jank on the
IOThread - when we post a OnStreamReadyCallback task and when it gets
run.

Original issue's description:
>QUIC - track the time it takes for OnStreamReadyCallback() to be called.
>This times tells when a QUIC stream could have been bound to the request
>vs when it was actually bound.
>
>  Net.HttpStreamFactory.Job.StreamReadyCallbackTime
>
>Will revert this change after getting some real world data.
>
>R=asvitkine@chromium.org, rch@chromium.org
>
>Committed: https://crrev.com/91e60f1b060cd5cf9720f009181804486dc5aec8
>Cr-Commit-Position: refs/heads/master@{#370725}

BUG=610404
R=rkaplow@chromium.org, rch@chromium.org

Committed: https://crrev.com/5d565d4671801a3db93cf71adf01d20e3a39fa9a
Cr-Commit-Position: refs/heads/master@{#411830}

[ramant (doing other things), 2016-08-12 22:44:24]

The CQ bit was checked by rtenneti@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 22:44:47]

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

[Alexei Svitkine (slow), 2016-08-12 22:45:34]

asvitkine@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Alexei Svitkine (slow), 2016-08-12 22:45:35]

https://codereview.chromium.org/2232313004/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2232313004/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:28820: -<histogram
name="Net.HttpStreamFactoryJob.StreamReadyCallbackTime" units="ms">
Please mark this as <obsolete> instead of removing it. Thanks!

[ramant (doing other things), 2016-08-12 22:50:04]

Thanks Alexei for the quick review. Marked the histogram as obsolete. PTAL.

https://codereview.chromium.org/2232313004/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (left):

https://codereview.chromium.org/2232313004/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:28820: -<histogram
name="Net.HttpStreamFactoryJob.StreamReadyCallbackTime" units="ms">
On 2016/08/12 22:45:35, Alexei Svitkine (very slow) wrote:
> Please mark this as <obsolete> instead of removing it. Thanks!

Done.

[Ryan Hamilton, 2016-08-12 22:50:53]

lgtm

[Alexei Svitkine (slow), 2016-08-12 22:58:38]

lgtm

[ramant (doing other things), 2016-08-12 22:59:23]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2016-08-12 22:59:53]

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-08-13 00:26:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-13 00:29:27]

Description was changed from

==========
Revert of QUIC - track the time it takes for OnStreamReadyCallback() to
be called. (patchset #1 of https://codereview.chromium.org/1618443002/ )

Reason for revert:
We have collected the data.

Manaually reverted the following CL that collected data on jank on the
IOThread - when we post a OnStreamReadyCallback task and when it gets
run.

Original issue's description:
>QUIC - track the time it takes for OnStreamReadyCallback() to be called.
>This times tells when a QUIC stream could have been bound to the request
>vs when it was actually bound.
>
>  Net.HttpStreamFactory.Job.StreamReadyCallbackTime
>
>Will revert this change after getting some real world data.
>
>R=asvitkine@chromium.org, rch@chromium.org
>
>Committed: https://crrev.com/91e60f1b060cd5cf9720f009181804486dc5aec8
>Cr-Commit-Position: refs/heads/master@{#370725}

BUG=610404
R=rkaplow@chromium.org, rch@chromium.org
==========

to

==========
Revert of QUIC - track the time it takes for OnStreamReadyCallback() to
be called. (patchset #1 of https://codereview.chromium.org/1618443002/ )

Reason for revert:
We have collected the data.

Manaually reverted the following CL that collected data on jank on the
IOThread - when we post a OnStreamReadyCallback task and when it gets
run.

Original issue's description:
>QUIC - track the time it takes for OnStreamReadyCallback() to be called.
>This times tells when a QUIC stream could have been bound to the request
>vs when it was actually bound.
>
>  Net.HttpStreamFactory.Job.StreamReadyCallbackTime
>
>Will revert this change after getting some real world data.
>
>R=asvitkine@chromium.org, rch@chromium.org
>
>Committed: https://crrev.com/91e60f1b060cd5cf9720f009181804486dc5aec8
>Cr-Commit-Position: refs/heads/master@{#370725}

BUG=610404
R=rkaplow@chromium.org, rch@chromium.org

Committed: https://crrev.com/5d565d4671801a3db93cf71adf01d20e3a39fa9a
Cr-Commit-Position: refs/heads/master@{#411830}
==========

[commit-bot: I haz the power, 2016-08-13 00:29:28]

Patchset 2 (id:??) landed as
https://crrev.com/5d565d4671801a3db93cf71adf01d20e3a39fa9a
Cr-Commit-Position: refs/heads/master@{#411830}