Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5773)

Unified Diff: chrome/browser/policy/configuration_policy_handler_chromeos.cc

Issue 9111022: Removed ConfigurationPolicyType and extended PolicyMap. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/configuration_policy_handler_chromeos.cc
diff --git a/chrome/browser/policy/configuration_policy_handler_chromeos.cc b/chrome/browser/policy/configuration_policy_handler_chromeos.cc
index 8c0f1d8ae4be8139ec26698f24e9fea9e920880f..dc98e93c515efc1c9a9f36b55b2a10404c714f71 100644
--- a/chrome/browser/policy/configuration_policy_handler_chromeos.cc
+++ b/chrome/browser/policy/configuration_policy_handler_chromeos.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
@@ -18,9 +18,9 @@
namespace policy {
NetworkConfigurationPolicyHandler::NetworkConfigurationPolicyHandler(
- ConfigurationPolicyType type,
+ const char* policy_name,
chromeos::NetworkUIData::ONCSource onc_source)
- : TypeCheckingPolicyHandler(type, Value::TYPE_STRING),
+ : TypeCheckingPolicyHandler(policy_name, Value::TYPE_STRING),
onc_source_(onc_source) {}
NetworkConfigurationPolicyHandler::~NetworkConfigurationPolicyHandler() {}
@@ -37,7 +37,7 @@ bool NetworkConfigurationPolicyHandler::CheckPolicySettings(
value->GetAsString(&onc_blob);
chromeos::OncNetworkParser parser(onc_blob, onc_source_);
if (!parser.parse_error().empty()) {
- errors->AddError(policy_type(),
+ errors->AddError(policy_name(),
IDS_POLICY_NETWORK_CONFIG_PARSE_ERROR,
parser.parse_error());
return false;
@@ -56,15 +56,14 @@ void NetworkConfigurationPolicyHandler::ApplyPolicySettings(
void NetworkConfigurationPolicyHandler::PrepareForDisplaying(
PolicyMap* policies) const {
- const Value* network_config = policies->Get(policy_type());
- if (!network_config)
+ const PolicyMap::Entry* entry = policies->Get(policy_name());
+ if (!entry)
return;
-
- Value* sanitized_config = SanitizeNetworkConfig(network_config);
+ Value* sanitized_config = SanitizeNetworkConfig(entry->value);
if (!sanitized_config)
sanitized_config = Value::CreateNullValue();
- policies->Set(policy_type(), sanitized_config);
+ policies->Set(policy_name(), entry->level, entry->scope, sanitized_config);
}
// static

Powered by Google App Engine
This is Rietveld 408576698