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

Unified Diff: chrome/browser/chromeos/login/existing_user_controller.cc

Issue 2692163003: cros: Add enterprise user session metrics (Closed)
Patch Set: update sparse histogram comment Created 3 years, 10 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
Index: chrome/browser/chromeos/login/existing_user_controller.cc
diff --git a/chrome/browser/chromeos/login/existing_user_controller.cc b/chrome/browser/chromeos/login/existing_user_controller.cc
index 40e3b8386c317a89d954657e3b7776fa520f8e10..2dbba6fd394b114953716b5543a4891c49e82c21 100644
--- a/chrome/browser/chromeos/login/existing_user_controller.cc
+++ b/chrome/browser/chromeos/login/existing_user_controller.cc
@@ -27,6 +27,7 @@
#include "chrome/browser/chromeos/login/auth/chrome_login_performer.h"
#include "chrome/browser/chromeos/login/easy_unlock/bootstrap_user_context_initializer.h"
#include "chrome/browser/chromeos/login/easy_unlock/bootstrap_user_flow.h"
+#include "chrome/browser/chromeos/login/enterprise_user_session_metrics.h"
#include "chrome/browser/chromeos/login/helper.h"
#include "chrome/browser/chromeos/login/session/user_session_manager.h"
#include "chrome/browser/chromeos/login/signin/oauth2_token_initializer.h"
@@ -738,6 +739,13 @@ void ExistingUserController::OnAuthSuccess(const UserContext& user_context) {
std::string() /* locale */));
}
ClearRecordedNames();
+
+ if (g_browser_process->platform_part()
+ ->browser_policy_connector_chromeos()
+ ->IsEnterpriseManaged()) {
+ enterprise_user_session_metrics::RecordSignInEvent(
+ user_context, last_login_attempt_was_auto_login_);
+ }
}
void ExistingUserController::OnProfilePrepared(Profile* profile,
@@ -1021,16 +1029,20 @@ void ExistingUserController::ResetAutoLoginTimer() {
void ExistingUserController::OnPublicSessionAutoLoginTimerFire() {
CHECK(auto_launch_ready_ && public_session_auto_login_account_id_.is_valid());
+ SigninSpecifics signin_specifics;
+ signin_specifics.is_auto_login = true;
Login(UserContext(user_manager::USER_TYPE_PUBLIC_ACCOUNT,
public_session_auto_login_account_id_),
- SigninSpecifics());
+ signin_specifics);
}
void ExistingUserController::OnArcKioskAutoLoginTimerFire() {
CHECK(auto_launch_ready_ && (arc_kiosk_auto_login_account_id_.is_valid()));
+ SigninSpecifics signin_specifics;
+ signin_specifics.is_auto_login = true;
Login(UserContext(user_manager::USER_TYPE_ARC_KIOSK_APP,
arc_kiosk_auto_login_account_id_),
- SigninSpecifics());
+ signin_specifics);
}
void ExistingUserController::StopAutoLoginTimer() {
@@ -1268,6 +1280,8 @@ void ExistingUserController::DoCompleteLogin(
void ExistingUserController::DoLogin(const UserContext& user_context,
const SigninSpecifics& specifics) {
+ last_login_attempt_was_auto_login_ = specifics.is_auto_login;
+
if (user_context.GetUserType() == user_manager::USER_TYPE_GUEST) {
if (!specifics.guest_mode_url.empty()) {
guest_mode_url_ = GURL(specifics.guest_mode_url);
« no previous file with comments | « chrome/browser/chromeos/login/existing_user_controller.h ('k') | chrome/browser/chromeos/login/signin_specifics.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698