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

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

Issue 10384145: Removed ConfigurationPolicyProvider::Provide(). (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebased Created 8 years, 7 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/config_dir_policy_provider.cc
diff --git a/chrome/browser/policy/config_dir_policy_provider.cc b/chrome/browser/policy/config_dir_policy_provider.cc
index e8cb43c4be762ac5619de77a4c4763306837d6e0..a98a28caf373ab29957188167caa28a36158144c 100644
--- a/chrome/browser/policy/config_dir_policy_provider.cc
+++ b/chrome/browser/policy/config_dir_policy_provider.cc
@@ -6,13 +6,14 @@
#include <algorithm>
#include <set>
+#include <string>
#include "base/file_path.h"
#include "base/file_util.h"
#include "base/json/json_file_value_serializer.h"
#include "base/logging.h"
#include "base/platform_file.h"
-#include "chrome/browser/policy/policy_map.h"
+#include "chrome/browser/policy/policy_bundle.h"
namespace policy {
@@ -24,7 +25,7 @@ ConfigDirPolicyProviderDelegate::ConfigDirPolicyProviderDelegate(
level_(level),
scope_(scope) {}
-PolicyMap* ConfigDirPolicyProviderDelegate::Load() {
+scoped_ptr<PolicyBundle> ConfigDirPolicyProviderDelegate::Load() {
// Enumerate the files and sort them lexicographically.
std::set<FilePath> files;
file_util::FileEnumerator file_enumerator(config_file_path(), false,
@@ -37,7 +38,7 @@ PolicyMap* ConfigDirPolicyProviderDelegate::Load() {
// The files are processed in reverse order because |MergeFrom| gives priority
// to existing keys, but the ConfigDirPolicyProvider gives priority to the
// last file in lexicographic order.
- PolicyMap* policy = new PolicyMap;
+ scoped_ptr<PolicyBundle> bundle(new PolicyBundle());
for (std::set<FilePath>::reverse_iterator config_file_iter = files.rbegin();
config_file_iter != files.rend(); ++config_file_iter) {
JSONFileValueSerializer deserializer(*config_file_iter);
@@ -58,10 +59,10 @@ PolicyMap* ConfigDirPolicyProviderDelegate::Load() {
}
PolicyMap file_policy;
file_policy.LoadFrom(dictionary_value, level_, scope_);
- policy->MergeFrom(file_policy);
+ bundle->Get(POLICY_DOMAIN_CHROME, std::string()).MergeFrom(file_policy);
}
- return policy;
+ return bundle.Pass();
}
base::Time ConfigDirPolicyProviderDelegate::GetLastModification() {
« no previous file with comments | « chrome/browser/policy/config_dir_policy_provider.h ('k') | chrome/browser/policy/config_dir_policy_provider_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698