diff options
| author | vadimt@chromium.org <vadimt@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-01 19:27:42 +0000 |
|---|---|---|
| committer | vadimt@chromium.org <vadimt@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-01 19:27:42 +0000 |
| commit | dcc02d95f0953b2c0cd15d99b22c0e6b4101b2e7 (patch) | |
| tree | a23ffb9eee3d99d06a40ad464735799eb122c147 /content/browser/geolocation/network_location_request.cc | |
| parent | 0d006ff992118ab00e1c2bd45e91a404be6e3127 (diff) | |
| download | chromium_src-dcc02d95f0953b2c0cd15d99b22c0e6b4101b2e7.zip chromium_src-dcc02d95f0953b2c0cd15d99b22c0e6b4101b2e7.tar.gz chromium_src-dcc02d95f0953b2c0cd15d99b22c0e6b4101b2e7.tar.bz2 | |
Adding histogram for a LBS request latency. See bug.
BUG=158787
Review URL: https://chromiumcodereview.appspot.com/11364009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165471 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/geolocation/network_location_request.cc')
| -rw-r--r-- | content/browser/geolocation/network_location_request.cc | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/content/browser/geolocation/network_location_request.cc b/content/browser/geolocation/network_location_request.cc index 75868d8..ab53aa5 100644 --- a/content/browser/geolocation/network_location_request.cc +++ b/content/browser/geolocation/network_location_request.cc @@ -9,6 +9,7 @@ #include "base/json/json_reader.h" #include "base/json/json_writer.h" +#include "base/metrics/histogram.h" #include "base/string_number_conversions.h" #include "base/utf_string_conversions.h" #include "base/values.h" @@ -100,6 +101,7 @@ bool NetworkLocationRequest::MakeRequest(const string16& access_token, net::LOAD_DO_NOT_SAVE_COOKIES | net::LOAD_DO_NOT_SEND_COOKIES | net::LOAD_DO_NOT_SEND_AUTH_DATA); + start_time_ = base::TimeTicks::Now(); url_fetcher_->Start(); return true; } @@ -126,6 +128,17 @@ void NetworkLocationRequest::OnURLFetchComplete( !status.is_success() || (response_code >= 500 && response_code < 600); url_fetcher_.reset(); + if (!server_error) { + const base::TimeDelta request_time = base::TimeTicks::Now() - start_time_; + + UMA_HISTOGRAM_CUSTOM_TIMES( + "Net.Wifi.LbsLatency", + request_time, + base::TimeDelta::FromMilliseconds(1), + base::TimeDelta::FromSeconds(10), + 100); + } + DCHECK(listener_); DVLOG(1) << "NetworkLocationRequest::Run() : Calling listener with position."; listener_->LocationResponseAvailable(position, server_error, access_token, |