arc: Consolidate IsArcAllowedForUser logic

components/user_manager/fake_user_manager.h

Index: components/user_manager/fake_user_manager.h
diff --git a/components/user_manager/fake_user_manager.h b/components/user_manager/fake_user_manager.h
index 221dc42d71a7b472681f737642e4f682d76ceab9..c2064f4927fb9f33006dae7da5275c48fa484e1e 100644
--- a/components/user_manager/fake_user_manager.h
+++ b/components/user_manager/fake_user_manager.h
@@ -32,13 +32,9 @@ class USER_MANAGER_EXPORT FakeUserManager : public UserManagerBase {
       const AccountId& account_id,
       bool is_affiliated);
 
-  // Calculates the user name hash and calls UserLoggedIn to login a user.
-  void LoginUser(const AccountId& account_id);

[xiyuan, 2017/05/17 21:36:14]

Stale code with no impl.

-
   // UserManager overrides.
   const user_manager::UserList& GetUsers() const override;
   user_manager::UserList GetUsersAllowedForMultiProfile() const override;
-  const user_manager::UserList& GetLoggedInUsers() const override;

[xiyuan, 2017/05/17 21:36:14]

Removed because it is the same as base class impl.

 
   // Set the user as logged in.
   void UserLoggedIn(const AccountId& account_id,
@@ -119,6 +115,7 @@ class USER_MANAGER_EXPORT FakeUserManager : public UserManagerBase {
 
   // UserManagerBase overrides:
   bool AreEphemeralUsersEnabled() const override;
+  void SetEphemeralUsersEnabled(bool enabled) override;
   const std::string& GetApplicationLocale() const override;
   PrefService* GetLocalState() const override;
   void HandleUserOAuthTokenStatusChange(