UserManager stack refactoring.

chrome/browser/chromeos/login/users/chrome_user_manager_interface.h

-// Copyright 2014 The Chromium Authors. All rights reserved.
+// Copyright 2015 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.
 
-#ifndef CHROME_BROWSER_CHROMEOS_LOGIN_USERS_CHROME_USER_MANAGER_H_
-#define CHROME_BROWSER_CHROMEOS_LOGIN_USERS_CHROME_USER_MANAGER_H_
+#ifndef CHROME_BROWSER_CHROMEOS_LOGIN_USERS_CHROME_USER_MANAGER_INTERFACE_H_
+#define CHROME_BROWSER_CHROMEOS_LOGIN_USERS_CHROME_USER_MANAGER_INTERFACE_H_
 
 #include "base/basictypes.h"
 #include "components/user_manager/user.h"
-#include "components/user_manager/user_manager_base.h"
-
-namespace base {
-class TaskRunner;
-}
+#include "components/user_manager/user_type.h"
 
 namespace chromeos {
 
 class MultiProfileUserController;
 class SupervisedUserManager;
 class UserFlow;
 class UserImageManager;
 
-// Chrome specific interface of the UserManager.
-class ChromeUserManager : public user_manager::UserManagerBase {
+// Chrome specific add-ons interface for the UserManager.
+class ChromeUserManagerInterface {
  public:
-  ChromeUserManager(scoped_refptr<base::TaskRunner> task_runner,
-                    scoped_refptr<base::TaskRunner> blocking_task_runner);
-  ~ChromeUserManager() override;
-
-  // Returns current ChromeUserManager or NULL if instance hasn't been
-  // yet initialized.
-  static ChromeUserManager* Get();
+  ChromeUserManagerInterface() {}
+  virtual ~ChromeUserManagerInterface() {}
 
   // Helper method for sorting out of user list only users that can create
   // supervised users.
   static user_manager::UserList GetUsersAllowedAsSupervisedUserManagers(

[Nikita (slow), 2015/01/20 16:39:28]

As discussed makes sense to leave in ChromeUserManager and call as
ChromeUserManager::GetUsersAllowedAsSupervisedUserManagers() from fake instance.

[merkulova, 2015/01/21 14:10:33]

Done.

-      const user_manager::UserList& user_list);
+      const user_manager::UserList& user_list) {
+    user_manager::UserList result;

[ygorshenin1, 2015/01/20 18:33:40]

Header files are usually used for declarations, not definitions, especially for
non-trivial methods.

[merkulova, 2015/01/21 14:10:33]

Done.

+    for (user_manager::User* user : user_list) {
+      if (user->GetType() == user_manager::USER_TYPE_REGULAR) {
+        result.push_back(user);
+      }
+    }
+    return result;
+  }
 
   virtual MultiProfileUserController* GetMultiProfileUserController() = 0;
   virtual UserImageManager* GetUserImageManager(const std::string& user_id) = 0;
   virtual SupervisedUserManager* GetSupervisedUserManager() = 0;
 
   // Method that allows to set |flow| for user identified by |user_id|.
   // Flow should be set before login attempt.
   // Takes ownership of the |flow|, |flow| will be deleted in case of login
   // failure.
   virtual void SetUserFlow(const std::string& user_id, UserFlow* flow) = 0;
 
   // Return user flow for current user. Returns instance of DefaultUserFlow if
   // no flow was defined for current user, or user is not logged in.
   // Returned value should not be cached.
   virtual UserFlow* GetCurrentUserFlow() const = 0;
 
   // Return user flow for user identified by |user_id|. Returns instance of
   // DefaultUserFlow if no flow was defined for user.
   // Returned value should not be cached.
   virtual UserFlow* GetUserFlow(const std::string& user_id) const = 0;
 
   // Resets user flow for user identified by |user_id|.
   virtual void ResetUserFlow(const std::string& user_id) = 0;
 
-  DISALLOW_COPY_AND_ASSIGN(ChromeUserManager);
+  // Returns list of users allowed for supervised user creation.
+  // Returns an empty list in cases when supervised user creation or adding new
+  // users is restricted.
+  virtual user_manager::UserList
+      GetUsersAllowedForSupervisedUsersCreation() const = 0;
+
+  DISALLOW_COPY_AND_ASSIGN(ChromeUserManagerInterface);
 };
 
 }  // namespace chromeos
 
-#endif  // CHROME_BROWSER_CHROMEOS_LOGIN_USERS_CHROME_USER_MANAGER_H_
+#endif  // CHROME_BROWSER_CHROMEOS_LOGIN_USERS_CHROME_USER_MANAGER_INTERFACE_H_