diff options
| author | megjablon <megjablon@chromium.org> | 2014-11-13 12:20:23 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-11-13 20:20:38 +0000 |
| commit | dae9a3422aa206788df58369402a094345b51648 (patch) | |
| tree | 89c26a01d23651df3a74894b786794c8ac513a8b | |
| parent | 409b8ba247d0157bba6b35a406c7264df5b9e660 (diff) | |
| download | chromium_src-dae9a3422aa206788df58369402a094345b51648.zip chromium_src-dae9a3422aa206788df58369402a094345b51648.tar.gz chromium_src-dae9a3422aa206788df58369402a094345b51648.tar.bz2 | |
Removing unnecessary UMA Net.IOThreadCreationToHttpRequestStart
This UMA is no longer being tracked.
BUG=432764
Review URL: https://codereview.chromium.org/719413003
Cr-Commit-Position: refs/heads/master@{#304069}
| -rw-r--r-- | chrome/browser/net/chrome_network_delegate.cc | 30 | ||||
| -rw-r--r-- | tools/metrics/histograms/histograms.xml | 3 |
2 files changed, 3 insertions, 30 deletions
diff --git a/chrome/browser/net/chrome_network_delegate.cc b/chrome/browser/net/chrome_network_delegate.cc index 4b4d618..48be213 100644 --- a/chrome/browser/net/chrome_network_delegate.cc +++ b/chrome/browser/net/chrome_network_delegate.cc @@ -218,14 +218,6 @@ void RecordPrecacheStatsOnUIThread(const GURL& url, precache_manager->RecordStatsForFetch(url, fetch_time, size, was_cached); } - -void RecordIOThreadToRequestStartOnUIThread( - const base::TimeTicks& request_start) { - DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); - base::TimeDelta request_lag = request_start - - g_browser_process->io_thread()->creation_time(); - UMA_HISTOGRAM_TIMES("Net.IOThreadCreationToHTTPRequestStart", request_lag); -} #endif // defined(OS_ANDROID) void ReportInvalidReferrerSend(const GURL& target_url, @@ -373,28 +365,6 @@ int ChromeNetworkDelegate::OnBeforeURLRequest( net::URLRequest* request, const net::CompletionCallback& callback, GURL* new_url) { -#if defined(OS_ANDROID) - // This UMA tracks the time to the first user-initiated request start, so - // only non-null profiles are considered. - if (first_request_ && profile_) { - bool record_timing = true; - if (data_reduction_proxy_params_) { - record_timing = - (request->url() != data_reduction_proxy_params_->probe_url()) && - (request->url() != data_reduction_proxy_params_->warmup_url()); - } - if (record_timing) { - first_request_ = false; - net::LoadTimingInfo timing_info; - request->GetLoadTimingInfo(&timing_info); - BrowserThread::PostTask( - BrowserThread::UI, FROM_HERE, - base::Bind(&RecordIOThreadToRequestStartOnUIThread, - timing_info.request_start)); - } - } -#endif // defined(OS_ANDROID) - #if defined(ENABLE_CONFIGURATION_POLICY) // TODO(joaodasilva): This prevents extensions from seeing URLs that are // blocked. However, an extension might redirect the request to another URL, diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml index 16190fb..44c6cb2 100644 --- a/tools/metrics/histograms/histograms.xml +++ b/tools/metrics/histograms/histograms.xml @@ -16642,6 +16642,9 @@ Therefore, the affected-histogram name has to have at least one dot in it. </histogram> <histogram name="Net.IOThreadCreationToHttpRequestStart" units="milliseconds"> + <obsolete> + Deprecated 11/2014. No longer tracked. + </obsolete> <owner>bengr@chromium.org</owner> <summary> Time from when the IOThread is created to when the first URL request is |