Remove enterprise device user report size limit.

chrome/browser/chromeos/policy/device_status_collector.cc

Index: chrome/browser/chromeos/policy/device_status_collector.cc
diff --git a/chrome/browser/chromeos/policy/device_status_collector.cc b/chrome/browser/chromeos/policy/device_status_collector.cc
index 1a983a4402a487c940b3c8d710d7c8c6f4994a9e..2625c4f15f5bdfb4b61efbe0d29fbb0fe8b9e3a0 100644
--- a/chrome/browser/chromeos/policy/device_status_collector.cc
+++ b/chrome/browser/chromeos/policy/device_status_collector.cc
@@ -73,7 +73,7 @@ int64 TimestampToDayKey(Time timestamp) {
 }
 
 // Maximum number of users to report.
-const int kMaxUserCount = 5;
+const int kMaxUserCount = 500;
 
 }  // namespace
 
@@ -430,7 +430,6 @@ void DeviceStatusCollector::GetNetworkInterfaces(
 void DeviceStatusCollector::GetUsers(em::DeviceStatusReportRequest* request) {
   policy::BrowserPolicyConnectorChromeOS* connector =
       g_browser_process->platform_part()->browser_policy_connector_chromeos();
-  bool found_managed_user = false;
   const chromeos::UserList& users = chromeos::UserManager::Get()->GetUsers();
   chromeos::UserList::const_iterator user;
   for (user = users.begin(); user != users.end(); ++user) {
@@ -443,16 +442,13 @@ void DeviceStatusCollector::GetUsers(em::DeviceStatusReportRequest* request) {
     if (connector->GetUserAffiliation(email) == USER_AFFILIATION_MANAGED) {
       device_user->set_type(em::DeviceUser::USER_TYPE_MANAGED);
       device_user->set_email(email);
-      found_managed_user = true;
     } else {
       device_user->set_type(em::DeviceUser::USER_TYPE_UNMANAGED);
       // Do not report the email address of unmanaged users.
     }
 
-    // Add only kMaxUserCount entries, unless no managed users are found in the
-    // first kMaxUserCount users. In that case, continue until at least one
-    // managed user is found.
-    if (request->user_size() >= kMaxUserCount && found_managed_user)
+    // Add only kMaxUserCount entries.
+    if (request->user_size() >= kMaxUserCount)

[Joao da Silva, 2014/07/07 09:43:23]

500 is an extreme limit for this... why not just remove this logic? The bug
doesn't have enough context and I may be missing something here.

       break;
   }
 }