Policy providers all get a SchemaRegistry to work with.

chrome/browser/policy/cloud/component_cloud_policy_store.h

 // Copyright (c) 2013 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.
 
 #ifndef CHROME_BROWSER_POLICY_CLOUD_COMPONENT_CLOUD_POLICY_STORE_H_
 #define CHROME_BROWSER_POLICY_CLOUD_COMPONENT_CLOUD_POLICY_STORE_H_
 
 #include <map>
-#include <set>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/non_thread_safe.h"
+#include "chrome/browser/policy/cloud/resource_cache.h"
 #include "chrome/browser/policy/policy_bundle.h"
 #include "components/policy/core/common/policy_namespace.h"
 
 namespace enterprise_management {
 class ExternalPolicyData;
 class PolicyData;
 class PolicyFetchResponse;
 }
 
 namespace policy {
 
-class ResourceCache;
-
 // Validates protobufs for external policy data, validates the data itself, and
 // caches both locally.
 class ComponentCloudPolicyStore : public base::NonThreadSafe {
  public:
   class Delegate {
    public:
     virtual ~Delegate();
 
     // Invoked whenever the policies served by policy() have changed, except
     // for the initial Load().
@@ skipping 46 matching lines @@
   // Returns false if |data| failed validation, otherwise returns true and the
   // data was stored in the cache.
   bool Store(const PolicyNamespace& ns,
              const std::string& serialized_policy_proto,
              const std::string& secure_hash,
              const std::string& data);
 
   // Deletes the storage of namespace |ns| and stops serving its policies.
   void Delete(const PolicyNamespace& ns);
 
-  // Deletes the storage of all components of |domain| that are not in
-  // |components_to_keep|, and stops serving their policies.
+  // Deletes the storage of all components of |domain| that pass then given

[bartfab (slow), 2013/11/05 15:53:04]

It is kind of weird that those which pass the test get deleted. As commented in
resource_cache.h itself, I think renaming the clean-up method and/or its
argument and/or its type will help to make this more understandable.

[Joao da Silva, 2013/11/07 13:15:00]

It has been renamed to SubkeyFilter.

+  // |test|, and stops serving their policies.
   void Purge(PolicyDomain domain,
-             const std::set<std::string>& components_to_keep);
+             const ResourceCache::KeyTest& test);
 
   // Validates |proto| and returns the corresponding policy namespace in |ns|,
   // and the parsed ExternalPolicyData in |payload|.
   // If |proto| validates successfully then its |payload| can be trusted, and
   // the data referenced there can be downloaded. A |proto| must be validated
   // before attempting to download the data, and before storing both.
   bool ValidatePolicy(
       scoped_ptr<enterprise_management::PolicyFetchResponse> proto,
       PolicyNamespace* ns,
       enterprise_management::ExternalPolicyData* payload);
@@ skipping 26 matching lines @@
 
   PolicyBundle policy_bundle_;
   std::map<PolicyNamespace, std::string> cached_hashes_;
 
   DISALLOW_COPY_AND_ASSIGN(ComponentCloudPolicyStore);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_CLOUD_COMPONENT_CLOUD_POLICY_STORE_H_