| Index: chrome/browser/policy/configuration_policy_reader.cc
|
| diff --git a/chrome/browser/policy/configuration_policy_reader.cc b/chrome/browser/policy/configuration_policy_reader.cc
|
| index 49d957cba44a0931319e730e1202e9a3a38840d6..1854969dfdb65ff73bd16362ed3c16ea0b38db6c 100644
|
| --- a/chrome/browser/policy/configuration_policy_reader.cc
|
| +++ b/chrome/browser/policy/configuration_policy_reader.cc
|
| @@ -15,6 +15,7 @@
|
| #include "chrome/browser/policy/browser_policy_connector.h"
|
| #include "chrome/browser/policy/configuration_policy_pref_store.h"
|
| #include "chrome/browser/policy/policy_map.h"
|
| +#include "policy/policy_constants.h"
|
|
|
| namespace policy {
|
|
|
| @@ -34,9 +35,6 @@ class ConfigurationPolicyStatusKeeper {
|
|
|
| private:
|
| typedef std::map<ConfigurationPolicyType, PolicyStatusInfo*> PolicyStatusMap;
|
| - typedef std::map<ConfigurationPolicyType, string16> PolicyNameMap;
|
| - typedef ConfigurationPolicyProvider::PolicyDefinitionList
|
| - PolicyDefinitionList;
|
|
|
| // Calls Provide() on the passed in |provider| to get policy values.
|
| void GetPoliciesFromProvider(ConfigurationPolicyProvider* provider);
|
| @@ -78,8 +76,7 @@ void ConfigurationPolicyStatusKeeper::GetPoliciesFromProvider(
|
|
|
| PolicyMap::const_iterator policy = policies->begin();
|
| for ( ; policy != policies->end(); ++policy) {
|
| - string16 name = PolicyStatus::GetPolicyName(policy->first);
|
| - DCHECK(!name.empty());
|
| + string16 name = ASCIIToUTF16(GetPolicyName(policy->first));
|
|
|
| // TODO(simo) actually determine whether the policy is a user or a device
|
| // one and whether the policy could be enforced or not once this information
|
| @@ -221,7 +218,7 @@ ListValue* PolicyStatus::GetPolicyStatusList(bool* any_policies_set) const {
|
|
|
| *any_policies_set = false;
|
| const PolicyDefinitionList* supported_policies =
|
| - ConfigurationPolicyPrefStore::GetChromePolicyDefinitionList();
|
| + GetChromePolicyDefinitionList();
|
| const PolicyDefinitionList::Entry* policy = supported_policies->begin;
|
| for ( ; policy != supported_policies->end; ++policy) {
|
| if (!AddPolicyFromReaders(policy->policy_type, result)) {
|
| @@ -245,32 +242,6 @@ ListValue* PolicyStatus::GetPolicyStatusList(bool* any_policies_set) const {
|
| return result;
|
| }
|
|
|
| -// static
|
| -string16 PolicyStatus::GetPolicyName(ConfigurationPolicyType policy_type) {
|
| - static std::map<ConfigurationPolicyType, string16> name_map;
|
| - static const ConfigurationPolicyProvider::PolicyDefinitionList*
|
| - supported_policies = NULL;
|
| -
|
| - if (!supported_policies) {
|
| - supported_policies =
|
| - ConfigurationPolicyPrefStore::GetChromePolicyDefinitionList();
|
| -
|
| - // Create mapping from ConfigurationPolicyTypes to actual policy names.
|
| - const ConfigurationPolicyProvider::PolicyDefinitionList::Entry* entry =
|
| - supported_policies->begin;
|
| - for ( ; entry != supported_policies->end; ++entry)
|
| - name_map[entry->policy_type] = ASCIIToUTF16(entry->name);
|
| - }
|
| -
|
| - std::map<ConfigurationPolicyType, string16>::const_iterator entry =
|
| - name_map.find(policy_type);
|
| -
|
| - if (entry == name_map.end())
|
| - return string16();
|
| -
|
| - return entry->second;
|
| -}
|
| -
|
| bool PolicyStatus::AddPolicyFromReaders(
|
| ConfigurationPolicyType policy, ListValue* list) const {
|
| DictionaryValue* mp_policy =
|
|
|