Refactored users of PolicySchema to use the new policy::Schema class.

chrome/browser/policy/policy_loader_mac.cc

 // 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.
 
 #include "chrome/browser/policy/policy_loader_mac.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/mac/foundation_util.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/path_service.h"
 #include "base/platform_file.h"
 #include "base/sequenced_task_runner.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/policy/external_data_fetcher.h"
 #include "chrome/browser/policy/policy_bundle.h"
 #include "chrome/browser/policy/policy_domain_descriptor.h"
 #include "chrome/browser/policy/policy_load_status.h"
 #include "chrome/browser/policy/policy_map.h"
 #include "chrome/browser/policy/preferences_mac.h"
-#include "components/policy/core/common/policy_schema.h"
+#include "components/policy/core/common/schema.h"
 #include "policy/policy_constants.h"
 
 using base::mac::CFCast;
 using base::ScopedCFTypeRef;
 
 namespace policy {
 
 namespace {
 
 // Callback function for CFDictionaryApplyFunction. |key| and |value| are an
@@ skipping 168 matching lines @@
         id_prefix + it_schema->first, it_schema->second, &policy);
     if (!policy.empty()) {
       bundle->Get(PolicyNamespace(descriptor->domain(), it_schema->first))
           .Swap(&policy);
     }
   }
 }
 
 void PolicyLoaderMac::LoadPolicyForComponent(
     const std::string& bundle_id_string,
-    const PolicySchema* schema,
+    Schema schema,
     PolicyMap* policy) {
   // TODO(joaodasilva): extensions may be registered in a PolicyDomainDescriptor
-  // without a PolicySchema, to allow a graceful update of the Legacy Browser
-  // Support extension on Windows. Remove this temporary check once that support
-  // is removed.
-  if (!schema)
+  // without a schema, to allow a graceful update of the Legacy Browser Support
+  // extension on Windows. Remove this check once that support is removed.
+  if (!schema.valid())
     return;
 
   base::ScopedCFTypeRef<CFStringRef> bundle_id(
       base::SysUTF8ToCFStringRef(bundle_id_string));
   preferences_->AppSynchronize(bundle_id);
 
-  const PolicySchemaMap* map = schema->GetProperties();
-  if (!map) {
-    NOTREACHED();
-    return;
-  }
-
-  for (PolicySchemaMap::const_iterator it = map->begin();
-       it != map->end(); ++it) {
+  for (Schema::Iterator it = schema.GetPropertiesIterator();
+       !it.IsAtEnd(); it.Advance()) {
     base::ScopedCFTypeRef<CFStringRef> pref_name(
-        base::SysUTF8ToCFStringRef(it->first));
+        base::SysUTF8ToCFStringRef(it.key()));
     base::ScopedCFTypeRef<CFPropertyListRef> value(
         preferences_->CopyAppValue(pref_name, bundle_id));
     if (!value.get())
       continue;
     bool forced =
         preferences_->AppValueIsForced(pref_name, bundle_id);
     PolicyLevel level = forced ? POLICY_LEVEL_MANDATORY :
                                  POLICY_LEVEL_RECOMMENDED;
     scoped_ptr<base::Value> policy_value(CreateValueFromProperty(value));
-    if (policy_value)
-      policy->Set(it->first, level, POLICY_SCOPE_USER,
+    if (policy_value) {
+      policy->Set(it.key(), level, POLICY_SCOPE_USER,
                   policy_value.release(), NULL);
+    }
   }
 }
 
 void PolicyLoaderMac::OnFileUpdated(const base::FilePath& path, bool error) {
   if (!error)
     Reload(false);
 }
 
 }  // namespace policy