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

Side by Side Diff: chrome/browser/chromeos/login/existing_user_controller.cc

Issue 2902373002: Add Login.PromptToLoginTime UMA metric (Closed)
Patch Set: Created 3 years, 6 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 unified diff | Download patch
« no previous file with comments | « no previous file | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/chromeos/login/existing_user_controller.h" 5 #include "chrome/browser/chromeos/login/existing_user_controller.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 499 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 // sure that the user's cryptohome is encrypted in ext4 dircrypto to run the 510 // sure that the user's cryptohome is encrypted in ext4 dircrypto to run the
511 // latest Android runtime. 511 // latest Android runtime.
512 UserContext new_user_context = user_context; 512 UserContext new_user_context = user_context;
513 new_user_context.SetIsForcingDircrypto( 513 new_user_context.SetIsForcingDircrypto(
514 ShouldForceDircrypto(new_user_context.GetAccountId())); 514 ShouldForceDircrypto(new_user_context.GetAccountId()));
515 login_performer_->PerformLogin(new_user_context, auth_mode); 515 login_performer_->PerformLogin(new_user_context, auth_mode);
516 RecordPasswordLoginEvent(new_user_context); 516 RecordPasswordLoginEvent(new_user_context);
517 } 517 }
518 SendAccessibilityAlert( 518 SendAccessibilityAlert(
519 l10n_util::GetStringUTF8(IDS_CHROMEOS_ACC_LOGIN_SIGNING_IN)); 519 l10n_util::GetStringUTF8(IDS_CHROMEOS_ACC_LOGIN_SIGNING_IN));
520 if (!time_init_.is_null()) {
521 base::TimeDelta delta = base::Time::Now() - time_init_;
522 UMA_HISTOGRAM_MEDIUM_TIMES("Login.PromptToLoginTime", delta);
523 time_init_ = base::Time(); // Reset to null.
524 }
520 } 525 }
521 526
522 void ExistingUserController::ContinuePerformLogin( 527 void ExistingUserController::ContinuePerformLogin(
523 LoginPerformer::AuthorizationMode auth_mode, 528 LoginPerformer::AuthorizationMode auth_mode,
524 const UserContext& user_context) { 529 const UserContext& user_context) {
525 login_performer_->PerformLogin(user_context, auth_mode); 530 login_performer_->PerformLogin(user_context, auth_mode);
526 } 531 }
527 532
528 void ExistingUserController::MigrateUserData(const std::string& old_password) { 533 void ExistingUserController::MigrateUserData(const std::string& old_password) {
529 // LoginPerformer instance has state of the user so it should exist. 534 // LoginPerformer instance has state of the user so it should exist.
(...skipping 963 matching lines...) Expand 10 before | Expand all | Expand 10 after
1493 login_display_->ShowUnrecoverableCrypthomeErrorDialog(); 1498 login_display_->ShowUnrecoverableCrypthomeErrorDialog();
1494 } 1499 }
1495 1500
1496 void ExistingUserController::ClearRecordedNames() { 1501 void ExistingUserController::ClearRecordedNames() {
1497 display_email_.clear(); 1502 display_email_.clear();
1498 display_name_.clear(); 1503 display_name_.clear();
1499 given_name_.clear(); 1504 given_name_.clear();
1500 } 1505 }
1501 1506
1502 } // namespace chromeos 1507 } // namespace chromeos
OLDNEW
« no previous file with comments | « no previous file | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698