diff options
author | pkasting <pkasting@chromium.org> | 2015-04-07 21:42:12 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-08 04:42:40 +0000 |
commit | 379234c28f3856045a67af657296d6911156bbff (patch) | |
tree | c0d7f0cab2fb491b97efec10fc8c444c49fc1ade /net/dns/host_resolver_impl.cc | |
parent | 8f0f3b63b8fd45d72ab957bdc12c714899199a72 (diff) | |
download | chromium_src-379234c28f3856045a67af657296d6911156bbff.zip chromium_src-379234c28f3856045a67af657296d6911156bbff.tar.gz chromium_src-379234c28f3856045a67af657296d6911156bbff.tar.bz2 |
Remove unnecessary instrumentation for several jank bugs.
BUG=437890,436671,455423,424386,436634,422516
TEST=none
TBR=jochen,cbentzel,jyasskin
Review URL: https://codereview.chromium.org/1062413002
Cr-Commit-Position: refs/heads/master@{#324185}
Diffstat (limited to 'net/dns/host_resolver_impl.cc')
-rw-r--r-- | net/dns/host_resolver_impl.cc | 31 |
1 files changed, 0 insertions, 31 deletions
diff --git a/net/dns/host_resolver_impl.cc b/net/dns/host_resolver_impl.cc index 9020476..e84a844 100644 --- a/net/dns/host_resolver_impl.cc +++ b/net/dns/host_resolver_impl.cc @@ -24,7 +24,6 @@ #include "base/message_loop/message_loop_proxy.h" #include "base/metrics/field_trial.h" #include "base/metrics/histogram.h" -#include "base/profiler/scoped_tracker.h" #include "base/stl_util.h" #include "base/strings/string_util.h" #include "base/strings/utf_string_conversions.h" @@ -495,11 +494,6 @@ class HostResolverImpl::Request { // Prepare final AddressList and call completion callback. void OnComplete(int error, const AddressList& addr_list) { - // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. - tracked_objects::ScopedTracker tracking_profile( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "436634 HostResolverImpl::Request::OnComplete")); - DCHECK(!was_canceled()); if (error == OK) *addresses_ = EnsurePortOnAddressList(addr_list, info_.port()); @@ -710,11 +704,6 @@ class HostResolverImpl::ProcTask const uint32 attempt_number, int error, const int os_error) { - // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. - tracked_objects::ScopedTracker tracking_profile1( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "436634 HostResolverImpl::ProcTask::OnLookupComplete1")); - DCHECK(origin_loop_->BelongsToCurrentThread()); // If results are empty, we should return an error. bool empty_list_on_ok = (error == OK && results.empty()); @@ -776,11 +765,6 @@ class HostResolverImpl::ProcTask net_log_.EndEvent(NetLog::TYPE_HOST_RESOLVER_IMPL_PROC_TASK, net_log_callback); - // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. - tracked_objects::ScopedTracker tracking_profile2( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "436634 HostResolverImpl::ProcTask::OnLookupComplete2")); - callback_.Run(error, results_); } @@ -1517,11 +1501,6 @@ class HostResolverImpl::Job : public PrioritizedDispatcher::Job, void OnProcTaskComplete(base::TimeTicks start_time, int net_error, const AddressList& addr_list) { - // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. - tracked_objects::ScopedTracker tracking_profile( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "436634 HostResolverImpl::Job::OnProcTaskComplete")); - DCHECK(is_proc_running()); if (!resolver_->resolved_known_ipv6_hostname_ && @@ -1671,11 +1650,6 @@ class HostResolverImpl::Job : public PrioritizedDispatcher::Job, // Performs Job's last rites. Completes all Requests. Deletes this. void CompleteRequests(const HostCache::Entry& entry, base::TimeDelta ttl) { - // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. - tracked_objects::ScopedTracker tracking_profile1( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "436634 HostResolverImpl::Job::CompleteRequests1")); - CHECK(resolver_.get()); // This job must be removed from resolver's |jobs_| now to make room for a @@ -1725,11 +1699,6 @@ class HostResolverImpl::Job : public PrioritizedDispatcher::Job, if (did_complete) resolver_->CacheResult(key_, entry, ttl); - // TODO(vadimt): Remove ScopedTracker below once crbug.com/436634 is fixed. - tracked_objects::ScopedTracker tracking_profile2( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "436634 HostResolverImpl::Job::CompleteRequests2")); - // Complete all of the requests that were attached to the job. for (RequestsList::const_iterator it = requests_.begin(); it != requests_.end(); ++it) { |