Instrumenting IOComplete internals to find jank.

Instrumenting IOComplete internals to find jank.
Instrumenting IOComplete internals to find jank.

From prior instrumentations, the jank turned to be in
QuicStreamFactory::Job::OnIOComplete - 26 jps
and
TransportConnectJobHelper::OnIOComplete - 7 jph

Instrumentation of the callback of QuicStreamFactory::Job::OnIOComplete is covered by CL https://codereview.chromium.org/789483004/.

This CL instruments the rest.

BUG=436634
Committed: https://crrev.com/da2e8a634f21c417d78a549d873c49ce7e7d1452
Cr-Commit-Position: refs/heads/master@{#308096}

[vadimt, 2014-12-10 23:59:03]

vadimt@chromium.org changed reviewers:
+ mmenke@chromium.org

[vadimt, 2014-12-10 23:59:04]

mmenke@, please review

[mmenke, 2014-12-11 20:00:38]

https://codereview.chromium.org/792233002/diff/1/net/quic/quic_stream_factory.cc
File net/quic/quic_stream_factory.cc (right):

https://codereview.chromium.org/792233002/diff/1/net/quic/quic_stream_factory...
net/quic/quic_stream_factory.cc:305: "436634
QuicStreamFactory::Job::OnIOComplete1"));
This is done in the other CL, using a different name.

[vadimt, 2014-12-11 20:22:26]

https://codereview.chromium.org/792233002/diff/1/net/quic/quic_stream_factory.cc
File net/quic/quic_stream_factory.cc (right):

https://codereview.chromium.org/792233002/diff/1/net/quic/quic_stream_factory...
net/quic/quic_stream_factory.cc:305: "436634
QuicStreamFactory::Job::OnIOComplete1"));
On 2014/12/11 20:00:38, mmenke wrote:
> This is done in the other CL, using a different name.

Done.

[mmenke, 2014-12-12 15:20:26]

LGTM

[mmenke, 2014-12-12 15:20:33]

LGTM

[vadimt, 2014-12-12 15:20:56]

The CQ bit was checked by vadimt@chromium.org

[commit-bot: I haz the power, 2014-12-12 15:21:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/792233002/20001

[commit-bot: I haz the power, 2014-12-12 16:21:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-12 16:23:30]

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