Put Kiosk App parameters into device settings.

chrome/browser/chromeos/login/user_manager_impl.cc

 // Copyright (c) 2013 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/user_manager_impl.h"
 
 #include <cstddef>
 #include <set>
 #include <vector>
 
@@ skipping 817 matching lines @@
       local_state->GetDictionary(kUserDisplayEmail);
 
   // Load regular users and locally managed users.
   std::vector<std::string> regular_users;
   std::set<std::string> regular_users_set;
   ParseUserList(*prefs_regular_users, std::set<std::string>(), "",
                 &regular_users, &regular_users_set);
   for (std::vector<std::string>::const_iterator it = regular_users.begin();
        it != regular_users.end(); ++it) {
     User* user = NULL;
-    if (gaia::ExtractDomainName(*it) ==
-            UserManager::kLocallyManagedUserDomain) {
+    const std::string domain_name = gaia::ExtractDomainName(*it);
+    if (domain_name == UserManager::kLocallyManagedUserDomain) {

[bartfab (slow), 2013/04/23 15:59:40]

Nit: Why |domain_name| here and |domain| 50 lines down? It would be nice to
settle on one name.

[Mattias Nissler (ping if slow), 2013/04/23 18:28:10]

Ha, this is an edit artifact actually :) Fixed.

       user = User::CreateLocallyManagedUser(*it);
     } else {
       user = User::CreateRegularUser(*it);
       user->set_oauth_token_status(LoadUserOAuthStatus(*it));
     }
     users_.push_back(user);
 
     string16 display_name;
     if (prefs_display_names->GetStringWithoutPathExpansion(*it,
                                                            &display_name)) {
@@ skipping 28 matching lines @@
   // Schedule a callback if device policy has not yet been verified.
   if (CrosSettingsProvider::TRUSTED != cros_settings_->PrepareTrustedValues(
       base::Bind(&UserManagerImpl::RetrieveTrustedDevicePolicies,
                  base::Unretained(this)))) {
     return;
   }
 
   cros_settings_->GetBoolean(kAccountsPrefEphemeralUsersEnabled,
                              &ephemeral_users_enabled_);
   cros_settings_->GetString(kDeviceOwner, &owner_email_);
-  const base::ListValue* public_accounts;
-  cros_settings_->GetList(kAccountsPrefDeviceLocalAccounts, &public_accounts);
+  const base::ListValue* device_local_accounts;
+  base::ListValue public_accounts;

[Nikita (slow), 2013/04/23 17:04:50]

nit: Could this be extracted to a separate method?

[Mattias Nissler (ping if slow), 2013/04/23 18:28:10]

Done.

+  if (cros_settings_->GetList(kAccountsPrefDeviceLocalAccounts,
+                              &device_local_accounts)) {
+    for (base::ListValue::const_iterator user(device_local_accounts->begin());
+         user != device_local_accounts->end(); ++user) {
+      std::string user_id;
+      if (!(*user)->GetAsString(&user_id))

[bartfab (slow), 2013/04/23 15:59:40]

Nit: Add NOTREACHED()?

[Mattias Nissler (ping if slow), 2013/04/23 18:28:10]

Done.

+        continue;
+      const std::string domain = gaia::ExtractDomainName(user_id);
+      if (domain == kKioskAppUserDomain) {
+        // TODO(mnissler, nkostylev, bartfab): Process Kiosk Apps within the
+        // standard user framework.

[Nikita (slow), 2013/04/23 17:04:50]

nit: Please file a bug.

[Mattias Nissler (ping if slow), 2013/04/23 18:28:10]

Done.

+        continue;
+      } else {
+        public_accounts.AppendString(user_id);
+      }
+    }
+  }
 
   EnsureUsersLoaded();
 
-  bool changed = UpdateAndCleanUpPublicAccounts(*public_accounts);
+  bool changed = UpdateAndCleanUpPublicAccounts(public_accounts);
 
   // If ephemeral users are enabled and we are on the login screen, take this
   // opportunity to clean up by removing all regular users except the owner.
   if (ephemeral_users_enabled_  && !IsUserLoggedIn()) {
     ListPrefUpdate prefs_users_update(g_browser_process->local_state(),
                                       kRegularUsers);
     prefs_users_update->Clear();
     for (UserList::iterator it = users_.begin(); it != users_.end(); ) {
       const std::string user_email = (*it)->email();
       if ((*it)->GetType() == User::USER_TYPE_REGULAR &&
@@ skipping 516 matching lines @@
     login_user_type = LoginState::LOGGED_IN_USER_LOCALLY_MANAGED;
   else if (active_user_->GetType() == User::USER_TYPE_KIOSK_APP)
     login_user_type = LoginState::LOGGED_IN_USER_KIOSK_APP;
   else
     login_user_type = LoginState::LOGGED_IN_USER_REGULAR;
 
   LoginState::Get()->SetLoggedInState(logged_in_state, login_user_type);
 }
 
 }  // namespace chromeos