diff options
| author | vadimt <vadimt@chromium.org> | 2014-11-04 08:53:11 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-11-04 16:53:37 +0000 |
| commit | c17c4cf24972a763a82230383fd40bd29620bab3 (patch) | |
| tree | cb2547e44329a9567d1ccb418f0818e1d6fbb581 /extensions/browser/updater | |
| parent | 16d4cbbb67e198ebad8ccb774e5ab53e6dc84303 (diff) | |
| download | chromium_src-c17c4cf24972a763a82230383fd40bd29620bab3.zip chromium_src-c17c4cf24972a763a82230383fd40bd29620bab3.tar.gz chromium_src-c17c4cf24972a763a82230383fd40bd29620bab3.tar.bz2 | |
Removing instrumentations for task URLFetcherCore::OnCompletedURLRequest
The jank almost disappeared, either because it was fixed, or its numbers were absorbed by other instrumentations.
At the current moment in Win, instrumentation for OAuth2AccessTokenFetcherImpl::OnURLFetchComplete shown 1.65 118ms jph, and remaining own jankiness of URLFetcherCore::OnCompletedURLRequest is 2.87.
BUG=422577
TBR=toyoshim@chromium.org, dconnelly@chromium.org, pkasting@chromium.org, zea@chromium.org, yoz@chromium.org, mmenke@chromium.org, dcheng@chromium.org, mattm@chromium.org, jhawkins@chromium.org, rogerta@chromium.org
Review URL: https://codereview.chromium.org/701463004
Cr-Commit-Position: refs/heads/master@{#302620}
Diffstat (limited to 'extensions/browser/updater')
| -rw-r--r-- | extensions/browser/updater/extension_downloader.cc | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/extensions/browser/updater/extension_downloader.cc b/extensions/browser/updater/extension_downloader.cc index a472758..3068027 100644 --- a/extensions/browser/updater/extension_downloader.cc +++ b/extensions/browser/updater/extension_downloader.cc @@ -13,7 +13,6 @@ #include "base/logging.h" #include "base/metrics/histogram.h" #include "base/metrics/sparse_histogram.h" -#include "base/profiler/scoped_tracker.h" #include "base/stl_util.h" #include "base/strings/string_number_conversions.h" #include "base/strings/string_util.h" @@ -492,11 +491,6 @@ void ExtensionDownloader::CreateManifestFetcher() { } void ExtensionDownloader::OnURLFetchComplete(const net::URLFetcher* source) { - // TODO(vadimt): Remove ScopedTracker below once crbug.com/422577 is fixed. - tracked_objects::ScopedTracker tracking_profile( - FROM_HERE_WITH_EXPLICIT_FUNCTION( - "422577 ExtensionDownloader::OnURLFetchComplete")); - VLOG(2) << source->GetResponseCode() << " " << source->GetURL(); if (source == manifest_fetcher_.get()) { |