diff options
author | vadimt <vadimt@chromium.org> | 2014-12-08 12:51:23 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-12-08 20:51:44 +0000 |
commit | 0e2715e4c66bf6af48be79ff351d6cce087f148b (patch) | |
tree | 13f35c033588936dda0fff4042e77e96e3913aed /net/quic/quic_stream_factory.cc | |
parent | 5d8bb1d2a56a634ed5dbc062912e52bf2bdcb2ba (diff) | |
download | chromium_src-0e2715e4c66bf6af48be79ff351d6cce087f148b.zip chromium_src-0e2715e4c66bf6af48be79ff351d6cce087f148b.tar.gz chromium_src-0e2715e4c66bf6af48be79ff351d6cce087f148b.tar.bz2 |
Instrumenting callbacks of SingleRequestHostResolver to find jank.
SingleRequestHostResolver::OnResolveCompletion is responsible for this bug's jank.
Instrumenting all its callbacks.
BUG=436634
Review URL: https://codereview.chromium.org/785633003
Cr-Commit-Position: refs/heads/master@{#307330}
Diffstat (limited to 'net/quic/quic_stream_factory.cc')
-rw-r--r-- | net/quic/quic_stream_factory.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/net/quic/quic_stream_factory.cc b/net/quic/quic_stream_factory.cc index 9625d0d..738326a 100644 --- a/net/quic/quic_stream_factory.cc +++ b/net/quic/quic_stream_factory.cc @@ -10,6 +10,7 @@ #include "base/message_loop/message_loop.h" #include "base/message_loop/message_loop_proxy.h" #include "base/metrics/histogram.h" +#include "base/profiler/scoped_tracker.h" #include "base/rand_util.h" #include "base/stl_util.h" #include "base/strings/string_util.h" @@ -291,6 +292,11 @@ int QuicStreamFactory::Job::DoLoop(int rv) { } void QuicStreamFactory::Job::OnIOComplete(int rv) { + // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. + tracked_objects::ScopedTracker tracking_profile( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "436634 QuicStreamFactory::Job::OnIOComplete")); + rv = DoLoop(rv); if (rv != ERR_IO_PENDING && !callback_.is_null()) { |