Explicitly forbid multiprofiles on the Active Directory devices.

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

 // Copyright 2014 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/login/users/chrome_user_manager_impl.h"
 
 #include <stddef.h>
 
 #include <cstddef>
 #include <set>
@@ skipping 280 matching lines @@
 }
 
 user_manager::UserList ChromeUserManagerImpl::GetUsersAllowedForMultiProfile()
     const {
   // Supervised users are not allowed to use multi-profiles.
   if (GetLoggedInUsers().size() == 1 &&
       GetPrimaryUser()->GetType() != user_manager::USER_TYPE_REGULAR) {
     return user_manager::UserList();
   }
 
+  // Multiprofiles are not allowed on the Active Directory managed devices.
+  policy::BrowserPolicyConnectorChromeOS* connector =
+      g_browser_process->platform_part()->browser_policy_connector_chromeos();
+  if (connector && connector->IsActiveDirectoryManaged())

[Thiemo Nagel, 2016/12/22 13:52:08]

In which case could the connector be nullptr?

[Roman Sorokin (ftl), 2016/12/22 13:56:05]

I dunno. Probably never could.

[Thiemo Nagel, 2016/12/22 13:57:49]

Then maybe we shouldn't check for it?

[Roman Sorokin (ftl), 2016/12/22 14:13:03]

Done.

+    return user_manager::UserList();
+
   user_manager::UserList result;
   const user_manager::UserList& users = GetUsers();
   for (user_manager::UserList::const_iterator it = users.begin();
        it != users.end();
        ++it) {
     if ((*it)->GetType() == user_manager::USER_TYPE_REGULAR &&
         !(*it)->is_logged_in()) {
       MultiProfileUserController::UserAllowedInSessionReason check;
       multi_profile_user_controller_->IsUserAllowedInSession(
           (*it)->GetAccountId().GetUserEmail(), &check);
@@ skipping 1046 matching lines @@
       break;
     default:
       NOTREACHED();
       break;
   }
 
   return user;
 }
 
 }  // namespace chromeos