New policy protobuf protocol.

chrome/browser/policy/configuration_policy_provider.h

Index: chrome/browser/policy/configuration_policy_provider.h
diff --git a/chrome/browser/policy/configuration_policy_provider.h b/chrome/browser/policy/configuration_policy_provider.h
index af2cddf914d8d30bc56ba1373b1d347513c38650..f8e57d81b9d87bb5639125b741d4a9a2182de579 100644
--- a/chrome/browser/policy/configuration_policy_provider.h
+++ b/chrome/browser/policy/configuration_policy_provider.h
@@ -6,15 +6,21 @@
 #define CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_PROVIDER_H_
 #pragma once
 
+#include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/browser/policy/configuration_policy_store_interface.h"
+// configuration_policy_type.h is generated. See policy_templates.json for
+// policy definitions.

[Mattias Nissler (ping if slow), 2011/02/02 12:27:55]

I'm no fan of repeating that comment over and over again. Let's just drop it,
people who are curious will learn by looking at the header of that file.

[Jakob Kummerow, 2011/02/03 14:36:52]

Done.

+#include "policy/configuration_policy_type.h"
 
 namespace policy {
 
+typedef std::map<ConfigurationPolicyType, Value*> PolicyMapType;

[Mattias Nissler (ping if slow), 2011/02/02 12:27:55]

That should really be a typedef inside ConfigurationPolicyProvider, no?

[Jakob Kummerow, 2011/02/03 14:36:52]

Done. As you wish.

+
 // A mostly-abstract super class for platform-specific policy providers.
 // Platform-specific policy providers (Windows Group Policy, gconf,
 // etc.) should implement a subclass of this class.
@@ -62,6 +68,11 @@ class ConfigurationPolicyProvider {
   void DecodePolicyValueTree(const DictionaryValue* policies,
                              ConfigurationPolicyStoreInterface* store);
 
+  // Writes the configuration found in the already-decoded map |policies| to
+  // the given |store|.
+  void ApplyPolicyMap(const PolicyMapType* policies,

[Mattias Nissler (ping if slow), 2011/02/02 12:27:55]

Can we name that similar to DecodePolicyValueTree? It does the same thing, just
on a different data structure.

[Jakob Kummerow, 2011/02/03 14:36:52]

Done.

+                      ConfigurationPolicyStoreInterface* store);
+
   const PolicyDefinitionList* policy_definition_list() const {
     return policy_definition_list_;
   }