diff options
| author | vadimt <vadimt@chromium.org> | 2014-12-02 09:37:49 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-12-02 17:38:23 +0000 |
| commit | 6eef6b86b1c6b2eb793abc895065acef9a443e1f (patch) | |
| tree | 594691649758818f3f3e9b2b60ad4876ab2b0f49 /extensions/browser/api/dns | |
| parent | 441a9c24faa811c0ed99cac957b34015c1bb803f (diff) | |
| download | chromium_src-6eef6b86b1c6b2eb793abc895065acef9a443e1f.zip chromium_src-6eef6b86b1c6b2eb793abc895065acef9a443e1f.tar.gz chromium_src-6eef6b86b1c6b2eb793abc895065acef9a443e1f.tar.bz2 | |
Instrumenting most popular callbacks of HostResolverImpl::Request::OnComplete.
HostResolverImpl::Request::OnComplete is the source of the jank for this bug (11.7 jph).
This CL doesn't instrument all callbacks, but 2 most popular (on my machine).
BUG=436634
Review URL: https://codereview.chromium.org/768233005
Cr-Commit-Position: refs/heads/master@{#306411}
Diffstat (limited to 'extensions/browser/api/dns')
| -rw-r--r-- | extensions/browser/api/dns/dns_api.cc | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/extensions/browser/api/dns/dns_api.cc b/extensions/browser/api/dns/dns_api.cc index ae61123..17bfdd7 100644 --- a/extensions/browser/api/dns/dns_api.cc +++ b/extensions/browser/api/dns/dns_api.cc @@ -5,6 +5,7 @@ #include "extensions/browser/api/dns/dns_api.h" #include "base/bind.h" +#include "base/profiler/scoped_tracker.h" #include "base/values.h" #include "content/public/browser/browser_context.h" #include "content/public/browser/browser_thread.h" @@ -80,6 +81,11 @@ void DnsResolveFunction::RespondOnUIThread() { } void DnsResolveFunction::OnLookupFinished(int resolve_result) { + // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. + tracked_objects::ScopedTracker tracking_profile( + FROM_HERE_WITH_EXPLICIT_FUNCTION( + "436634 DnsResolveFunction::OnLookupFinished")); + scoped_ptr<ResolveCallbackResolveInfo> resolve_info( new ResolveCallbackResolveInfo()); resolve_info->result_code = resolve_result; |