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

Unified Diff: chrome/browser/lifetime/application_lifetime.cc

Issue 180243025: Changing the UsersPerSession UMA metric to only count possible multi profile sessions and do this u… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 9 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/lifetime/application_lifetime.cc
diff --git a/chrome/browser/lifetime/application_lifetime.cc b/chrome/browser/lifetime/application_lifetime.cc
index 8af4ecd15a3a58a860af699f309974c187a74ab3..f770579866d1d965480f3ccb34280b72fdf11119 100644
--- a/chrome/browser/lifetime/application_lifetime.cc
+++ b/chrome/browser/lifetime/application_lifetime.cc
@@ -4,7 +4,6 @@
#include "chrome/browser/lifetime/application_lifetime.h"
-#include "ash/shell.h"
#include "base/bind.h"
#include "base/command_line.h"
#include "base/debug/trace_event.h"
@@ -36,8 +35,6 @@
#include "content/public/browser/notification_service.h"
#if defined(OS_CHROMEOS)
-#include "ash/multi_profile_uma.h"
-#include "ash/session_state_delegate.h"
#include "base/sys_info.h"
#include "chrome/browser/chromeos/boot_times_loader.h"
#include "chrome/browser/chromeos/login/user_manager.h"
@@ -150,12 +147,6 @@ void AttemptUserExit() {
const char kLogoutStarted[] = "logout-started";
chromeos::BootTimesLoader::Get()->RecordCurrentStats(kLogoutStarted);
- // Since we are shutting down now we should record how many users have joined
- // the session since session start.
- ash::MultiProfileUMA::RecordUserCount(
- ash::Shell::GetInstance()->session_state_delegate()->
- NumberOfLoggedInUsers());
-
// Login screen should show up in owner's locale.
PrefService* state = g_browser_process->local_state();
if (state) {

Powered by Google App Engine
This is Rietveld 408576698