Creating supervised users restricted for regular users if they are supervised.

chrome/browser/chromeos/login/users/chrome_user_manager.cc

Index: chrome/browser/chromeos/login/users/chrome_user_manager.cc
diff --git a/chrome/browser/chromeos/login/users/chrome_user_manager.cc b/chrome/browser/chromeos/login/users/chrome_user_manager.cc
index 1800e302f24de26cf725dd50efe530e82a60564d..54dae77df43b22d8cdabb1c5671c4b08839e5bee 100644
--- a/chrome/browser/chromeos/login/users/chrome_user_manager.cc
+++ b/chrome/browser/chromeos/login/users/chrome_user_manager.cc
@@ -22,4 +22,16 @@ ChromeUserManager* ChromeUserManager::Get() {
   return user_manager ? static_cast<ChromeUserManager*>(user_manager) : NULL;
 }
 
+// static
+user_manager::UserList ChromeUserManager::GetUsersAllowedAsSUManagers(
+    const user_manager::UserList& user_list) {
+  user_manager::UserList result;
+  for (user_manager::User* user : user_list) {
+    if (user->GetType() == user_manager::USER_TYPE_REGULAR) {

[Nikita (slow), 2014/11/12 11:09:08]

nit: merge with https://codereview.chromium.org/718673002/, 
may use here
if (user->IsRegular() && !user->IsSupervised())

[merkulova, 2014/11/12 13:19:51]

Why not simply leave this check?

[Nikita (slow), 2014/11/12 13:43:37]

I think that this condition (user->IsRegular() && !user->IsSupervised()) is more
self-descriptive.

Looking at (user->GetType() == user_manager::USER_TYPE_REGULAR) one can easily
forget that there's also USER_TYPE_REGULAR_SUPERVISED.

Plus condition that I propose is shorter and doesn't expose enum value which
should be avoid if possible.

+      result.push_back(user);
+    }
+  }
+  return result;
+}
+
 }  // namespace chromeos