arc: Consolidate IsArcAllowedForUser logic

chrome/browser/chromeos/login/screens/user_selection_screen.cc

Index: chrome/browser/chromeos/login/screens/user_selection_screen.cc
diff --git a/chrome/browser/chromeos/login/screens/user_selection_screen.cc b/chrome/browser/chromeos/login/screens/user_selection_screen.cc
index 3a5a6a212ef7f2418101f0966af13c207f386fde..7678705dbb03c1847712689540dfccb20e72c551 100644
--- a/chrome/browser/chromeos/login/screens/user_selection_screen.cc
+++ b/chrome/browser/chromeos/login/screens/user_selection_screen.cc
@@ -151,15 +151,9 @@ bool ShouldCheckNeedDircryptoMigration() {
 
 // Returns true if the user can run ARC based on the user type.
 bool IsUserAllowedForARC(const AccountId& account_id) {
-  if (!user_manager::UserManager::IsInitialized())
-    return false;
-
-  const user_manager::User* user =
-      user_manager::UserManager::Get()->FindUser(account_id);
-  if (!user)
-    return false;
-
-  return user->HasGaiaAccount() || user->IsActiveDirectoryUser();
+  return user_manager::UserManager::IsInitialized() &&
+         arc::IsArcAllowedForUser(
+             user_manager::UserManager::Get()->FindUser(account_id));

[hidehiko, 2017/05/23 07:23:26]

Clarification: Just in case, this change make behavior difference in case of
Kiosk, IIUC. Is it intentional?

[xiyuan, 2017/05/24 19:22:56]

It is okay because the function is called for user pods banners. Kiosk users do
not have pods on the login screen. So the behvaior change should be no impact on
kiosk users.

 }
 
 }  // namespace