Add device policy definition for local accounts.

chrome/browser/chromeos/settings/device_settings_provider.cc

 // Copyright (c) 2012 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/settings/device_settings_provider.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/file_util.h"
@@ skipping 22 matching lines @@
 
 namespace {
 
 // List of settings handled by the DeviceSettingsProvider.
 const char* kKnownSettings[] = {
   kAccountsPrefAllowGuest,
   kAccountsPrefAllowNewUser,
   kAccountsPrefEphemeralUsersEnabled,
   kAccountsPrefShowUserNamesOnSignIn,
   kAccountsPrefUsers,
+  kAccountsPrefLocalAccounts,
   kAppPack,
   kDeviceOwner,
   kIdleLogoutTimeout,
   kIdleLogoutWarningDuration,
   kPolicyMissingMitigationMode,
   kReleaseChannel,
   kReleaseChannelDelegated,
   kReportDeviceActivityTimes,
   kReportDeviceBootMode,
   kReportDeviceLocation,
@@ skipping 151 matching lines @@
     else
       NOTREACHED();
   } else if (prop == kAccountsPrefShowUserNamesOnSignIn) {
     em::ShowUserNamesOnSigninProto* show =
         device_settings_.mutable_show_user_names();
     bool show_value;
     if (value->GetAsBoolean(&show_value))
       show->set_show_user_names(show_value);
     else
       NOTREACHED();
+  } else if (prop == kAccountsPrefLocalAccounts) {
+    em::LocalAccountsProto * local_accounts =

[pastarmovj, 2012/11/06 12:42:15]

one space too much.

[Mattias Nissler (ping if slow), 2012/11/12 15:50:51]

Done.

+        device_settings_.mutable_local_accounts();
+    base::ListValue* accounts_list;
+    if (value->GetAsList(&accounts_list)) {
+      for (base::ListValue::const_iterator entry(accounts_list->begin());
+           entry != accounts_list->end(); ++entry) {
+        base::DictionaryValue* entry_dict;
+        std::string id;
+        std::string name;
+        if (!(*entry)->GetAsDictionary(&entry_dict) ||

[pastarmovj, 2012/11/06 12:42:15]

Isn't the logic inverted here?
If not read then set in the proto_entry else not reached is how i read it now?

[Mattias Nissler (ping if slow), 2012/11/12 15:50:51]

Done.

+            !entry_dict->GetString(kLocalAccountId, &id) ||
+            !entry_dict->GetString(kLocalAccountName, &name)) {
+          em::LocalAccountDefinitionProto* proto_entry =
+              local_accounts->add_local_account();
+          proto_entry->set_id(id);
+          proto_entry->set_name(name);
+        } else {
+          NOTREACHED();
+        }
+      }
+    } else {
+      NOTREACHED();
+    }
   } else if (prop == kSignedDataRoamingEnabled) {
     em::DataRoamingEnabledProto* roam =
         device_settings_.mutable_data_roaming_enabled();
     bool roaming_value = false;
     if (value->GetAsBoolean(&roaming_value))
       roam->set_data_roaming_enabled(roaming_value);
     else
       NOTREACHED();
     ApplyRoamingSetting(roaming_value);
   } else if (prop == kSettingProxyEverywhere) {
@@ skipping 130 matching lines @@
 
   base::ListValue* list = new base::ListValue();
   const em::UserWhitelistProto& whitelist_proto = policy.user_whitelist();
   const RepeatedPtrField<std::string>& whitelist =
       whitelist_proto.user_whitelist();
   for (RepeatedPtrField<std::string>::const_iterator it = whitelist.begin();
        it != whitelist.end(); ++it) {
     list->Append(base::Value::CreateStringValue(*it));
   }
   new_values_cache->SetValue(kAccountsPrefUsers, list);
+
+  base::ListValue* account_list = new base::ListValue();
+  const RepeatedPtrField<em::LocalAccountDefinitionProto>& local_accounts =
+      policy.local_accounts().local_account();
+  RepeatedPtrField<em::LocalAccountDefinitionProto>::const_iterator entry;
+  for (entry = local_accounts.begin(); entry != local_accounts.end(); ++entry) {
+    if (entry->has_id() && entry->has_name()) {
+      base::DictionaryValue* account_dict = new base::DictionaryValue();
+      account_dict->SetString(kLocalAccountId, entry->id());
+      account_dict->SetString(kLocalAccountName, entry->name());
+      account_list->Append(account_dict);
+    }
+  }
+  new_values_cache->SetValue(kAccountsPrefLocalAccounts, account_list);
 }
 
 void DeviceSettingsProvider::DecodeKioskPolicies(
     const em::ChromeDeviceSettingsProto& policy,
     PrefValueMap* new_values_cache) const {
   if (policy.has_forced_logout_timeouts()) {
     if (policy.forced_logout_timeouts().has_idle_logout_timeout()) {
       new_values_cache->SetInteger(
           kIdleLogoutTimeout,
           policy.forced_logout_timeouts().idle_logout_timeout());
@@ skipping 361 matching lines @@
 void DeviceSettingsProvider::AttemptMigration() {
   if (device_settings_service_->HasPrivateOwnerKey()) {
     PrefValueMap::const_iterator i;
     for (i = migration_values_.begin(); i != migration_values_.end(); ++i)
       DoSet(i->first, *i->second);
     migration_values_.Clear();
   }
 }
 
 }  // namespace chromeos