Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(785)

Unified Diff: chrome/browser/ui/webui/options/browser_options_handler.cc

Issue 22409002: Remove managed user registration timeout. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/managed_mode/managed_user_registration_utility.cc ('k') | chrome/common/chrome_switches.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/options/browser_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/browser_options_handler.cc b/chrome/browser/ui/webui/options/browser_options_handler.cc
index 09946702e4decb239c4f07a6df58e2fdbd0d9006..c45b91586dd2a9f2d284b533584dee4c8c745525 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -1183,11 +1183,9 @@ void BrowserOptionsHandler::RecordProfileCreationMetrics(
UMA_HISTOGRAM_ENUMERATION("Profile.CreateResult",
status,
Profile::MAX_CREATE_STATUS);
- UMA_HISTOGRAM_CUSTOM_TIMES("Profile.CreateTime",
- base::TimeTicks::Now() - profile_creation_start_time_,
- base::TimeDelta::FromMilliseconds(1),
- base::TimeDelta::FromSeconds(30), // From kRegistrationTimeoutMS.
- 100);
+ UMA_HISTOGRAM_MEDIUM_TIMES(
+ "Profile.CreateTimeNoTimeout",
+ base::TimeTicks::Now() - profile_creation_start_time_);
}
void BrowserOptionsHandler::ShowProfileCreationFeedback(
@@ -1298,11 +1296,9 @@ void BrowserOptionsHandler::CancelProfileRegistration(bool user_initiated) {
return;
if (user_initiated) {
- UMA_HISTOGRAM_CUSTOM_TIMES("Profile.CreateTimeCanceled",
- base::TimeTicks::Now() - profile_creation_start_time_,
- base::TimeDelta::FromMilliseconds(1),
- base::TimeDelta::FromSeconds(30), // From kRegistrationTimeoutMS.
- 100);
+ UMA_HISTOGRAM_MEDIUM_TIMES(
+ "Profile.CreateTimeCanceledNoTimeout",
+ base::TimeTicks::Now() - profile_creation_start_time_);
RecordProfileCreationMetrics(Profile::CREATE_STATUS_CANCELED);
}
« no previous file with comments | « chrome/browser/managed_mode/managed_user_registration_utility.cc ('k') | chrome/common/chrome_switches.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698