Add multi_profile_user_count field to system_profile proto.

chrome/browser/metrics/metrics_log.cc

Index: chrome/browser/metrics/metrics_log.cc
diff --git a/chrome/browser/metrics/metrics_log.cc b/chrome/browser/metrics/metrics_log.cc
index f2962a5015a797296396ea2cb0afa8db2c23b025..69ac8eb739a2d0f78b2efa60a5497022f4bf3e5d 100644
--- a/chrome/browser/metrics/metrics_log.cc
+++ b/chrome/browser/metrics/metrics_log.cc
@@ -62,6 +62,10 @@
 extern "C" IMAGE_DOS_HEADER __ImageBase;
 #endif
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/login/user_manager.h"
+#endif
+
 using content::GpuDataManager;
 using metrics::OmniboxEventProto;
 using metrics::PerfDataProto;
@@ -379,7 +383,11 @@ static base::LazyInstance<std::string>::Leaky
   g_version_extension = LAZY_INSTANCE_INITIALIZER;
 
 MetricsLog::MetricsLog(const std::string& client_id, int session_id)
-    : MetricsLogBase(client_id, session_id, MetricsLog::GetVersionString()) {}
+    : MetricsLogBase(client_id, session_id, MetricsLog::GetVersionString()) {
+#if defined(OS_CHROMEOS)
+  UpdateMultiProfileUserCount();
+#endif
+}
 
 MetricsLog::~MetricsLog() {}
 
@@ -799,6 +807,7 @@ void MetricsLog::RecordEnvironmentProto(
                  base::Unretained(this)));
   DCHECK(adapter_.get());
   WriteBluetoothProto(hardware);
+  UpdateMultiProfileUserCount();
 #endif
 }
 
@@ -954,3 +963,20 @@ void MetricsLog::WriteBluetoothProto(
   }
 #endif  // defined(OS_CHROMEOS)
 }
+
+#if defined(OS_CHROMEOS)
+void MetricsLog::UpdateMultiProfileUserCount() {
+  if (chromeos::UserManager::IsInitialized() &&
+      chromeos::UserManager::Get()->IsMultipleProfilesAllowed()) {
+    uint32 user_count = chromeos::UserManager::Get()
+        ->GetLoggedInUsers().size();
+    SystemProfileProto* system_profile = uma_proto()->mutable_system_profile();
+
+    // We invalidate the user count if it changed while the log was open.
+    if (system_profile->has_multi_profile_user_count() &&
+        user_count != system_profile->multi_profile_user_count())
+      user_count = 0;

[Ilya Sherman, 2013/10/08 01:31:03]

nit: Please leave a blank line after this one, so that it's easier to tell when
scanning that the below line is not part of the if-stmt.

[Tim Song, 2013/10/08 20:29:40]

Done.

+    system_profile->set_multi_profile_user_count(user_count);
+  }
+}
+#endif