Remove enterprise device user report size limit.

chrome/browser/chromeos/policy/device_status_collector_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/policy/device_status_collector.h"
 
 #include "base/environment.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 630 matching lines @@
   user_manager_->CreatePublicAccountUser("public@localhost");
   user_manager_->AddUser("user0@managed.com");
   user_manager_->AddUser("user1@managed.com");
   user_manager_->AddUser("user2@managed.com");
   user_manager_->AddUser("user3@unmanaged.com");
   user_manager_->AddUser("user4@managed.com");
   user_manager_->AddUser("user5@managed.com");
 
   // Verify that users are reported by default.
   GetStatus();
-  EXPECT_EQ(5, status_.user_size());
+  EXPECT_EQ(6, status_.user_size());
 
   // Verify that users are reported after enabling the setting.
   cros_settings_->SetBoolean(chromeos::kReportDeviceUsers, true);
   GetStatus();
-  EXPECT_EQ(5, status_.user_size());
+  EXPECT_EQ(6, status_.user_size());
   EXPECT_EQ(em::DeviceUser::USER_TYPE_MANAGED, status_.user(0).type());
   EXPECT_EQ("user0@managed.com", status_.user(0).email());
   EXPECT_EQ(em::DeviceUser::USER_TYPE_MANAGED, status_.user(1).type());
   EXPECT_EQ("user1@managed.com", status_.user(1).email());
   EXPECT_EQ(em::DeviceUser::USER_TYPE_MANAGED, status_.user(2).type());
   EXPECT_EQ("user2@managed.com", status_.user(2).email());
   EXPECT_EQ(em::DeviceUser::USER_TYPE_UNMANAGED, status_.user(3).type());
   EXPECT_FALSE(status_.user(3).has_email());
   EXPECT_EQ(em::DeviceUser::USER_TYPE_MANAGED, status_.user(4).type());
   EXPECT_EQ("user4@managed.com", status_.user(4).email());
+  EXPECT_EQ(em::DeviceUser::USER_TYPE_MANAGED, status_.user(5).type());
+  EXPECT_EQ("user5@managed.com", status_.user(5).email());
 
   // Verify that users are no longer reported if setting is disabled.
   cros_settings_->SetBoolean(chromeos::kReportDeviceUsers, false);
   GetStatus();
   EXPECT_EQ(0, status_.user_size());
 }
 
-TEST_F(DeviceStatusCollectorTest, ReportManagedUser) {
-  // Verify that at least one managed user is reported regardless of list size.
-  user_manager_->AddUser("user0@unmanaged.com");
-  user_manager_->AddUser("user1@unmanaged.com");
-  user_manager_->AddUser("user2@unmanaged.com");
-  user_manager_->AddUser("user3@unmanaged.com");
-  user_manager_->AddUser("user4@unmanaged.com");
-  user_manager_->AddUser("user5@unmanaged.com");
-  user_manager_->AddUser("user6@managed.com");
-  user_manager_->AddUser("user7@managed.com");
-
-  cros_settings_->SetBoolean(chromeos::kReportDeviceUsers, true);
-  GetStatus();
-  EXPECT_EQ(7, status_.user_size());
-  for (int i = 0; i < 6; ++i)
-    EXPECT_EQ(em::DeviceUser::USER_TYPE_UNMANAGED, status_.user(i).type());
-  EXPECT_EQ(em::DeviceUser::USER_TYPE_MANAGED, status_.user(6).type());
-  EXPECT_EQ("user6@managed.com", status_.user(6).email());
-}
-
 // Fake device state.
 struct FakeDeviceData {
   const char* device_path;
   const char* type;
   const char* object_path;
   const char* mac_address;
   const char* meid;
   const char* imei;
   int expected_type; // proto enum type value, -1 for not present.
 };
@@ skipping 101 matching lines @@
     }
 
     EXPECT_TRUE(found_match) << "No matching interface for fake device " << i;
     count++;
   }
 
   EXPECT_EQ(count, status_.network_interface_size());
 }
 
 }  // namespace policy