Policy providers all get a SchemaRegistry to work with.

chrome/browser/policy/policy_loader_mac.h

 // 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.
 
 #ifndef CHROME_BROWSER_POLICY_POLICY_LOADER_MAC_H_
 #define CHROME_BROWSER_POLICY_POLICY_LOADER_MAC_H_
 
 #include <string>
 
 #include <CoreFoundation/CoreFoundation.h>
 
 #include "base/files/file_path.h"
 #include "base/files/file_path_watcher.h"
 #include "base/memory/ref_counted.h"
 #include "chrome/browser/policy/async_policy_loader.h"
 
 class MacPreferences;
 
 namespace base {
 class SequencedTaskRunner;
 class Value;
 }  // namespace base
 
 namespace policy {
 
-class PolicyDomainDescriptor;
 class PolicyMap;
 class Schema;
 struct PolicyDefinitionList;
 
 // A policy loader that loads policies from the Mac preferences system, and
 // watches the managed preferences files for updates.
 class PolicyLoaderMac : public AsyncPolicyLoader {
  public:
   PolicyLoaderMac(scoped_refptr<base::SequencedTaskRunner> task_runner,
                   const PolicyDefinitionList* policy_list,
                   const base::FilePath& managed_policy_path,
                   MacPreferences* preferences);
   virtual ~PolicyLoaderMac();
 
   // AsyncPolicyLoader implementation.
   virtual void InitOnBackgroundThread() OVERRIDE;
   virtual scoped_ptr<PolicyBundle> Load() OVERRIDE;
   virtual base::Time LastModificationTime() OVERRIDE;
 
   // Converts a CFPropertyListRef to the equivalent base::Value. CFDictionary
   // entries whose key is not a CFStringRef are ignored.
   // The returned value is owned by the caller.
   // Returns NULL if an invalid CFType was found, such as CFDate or CFData.
   static base::Value* CreateValueFromProperty(CFPropertyListRef property);
 
  private:
   // Callback for the FilePathWatcher.
   void OnFileUpdated(const base::FilePath& path, bool error);
 
-  // Loads policies for the components described in |descriptor|, which belong
-  // to the domain |domain_name|, and stores them in the |bundle|.
+  // Loads policies for the components described in the current SchemaMap,

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

Nit 1: s/SchemaMap/schema_map_/ (reference the member, not its type).
Nit 2: The comma is wrong. It should not be here.

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

Done.

+  // which belong to the domain |domain_name|, and stores them in the |bundle|.
   void LoadPolicyForDomain(
-      scoped_refptr<const PolicyDomainDescriptor> descriptor,
+      PolicyDomain domain,

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

Nit: #include "components/policy/core/common/policy_namespace.h"

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

Done.

       const std::string& domain_name,
       PolicyBundle* bundle);

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

Nit: Forward-declare PolicyBundle.

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

Done.

 
   // Loads the policies described in |schema| from the bundle identified by
   // |bundle_id_string|, and stores them in |policy|.
   void LoadPolicyForComponent(const std::string& bundle_id_string,
-                              Schema schema,
+                              const Schema& schema,
                               PolicyMap* policy);
 
   // List of recognized policies.
   const PolicyDefinitionList* policy_list_;
 
   scoped_ptr<MacPreferences> preferences_;
 
   // Path to the managed preferences file for the current user, if it could
   // be found. Updates of this file trigger a policy reload.
   base::FilePath managed_policy_path_;
 
   // Watches for events on the |managed_policy_path_|.
   base::FilePathWatcher watcher_;
 
   DISALLOW_COPY_AND_ASSIGN(PolicyLoaderMac);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_POLICY_LOADER_MAC_H_