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

chrome/browser/ui/webui/policy_ui.cc

Index: chrome/browser/ui/webui/policy_ui.cc
diff --git a/chrome/browser/ui/webui/policy_ui.cc b/chrome/browser/ui/webui/policy_ui.cc
index fa9ed74dd23609b169d4344e39f2b6a8bfa8b5a3..f52d0ddebc10b33df7bf808f5141206fd515f73c 100644
--- a/chrome/browser/ui/webui/policy_ui.cc
+++ b/chrome/browser/ui/webui/policy_ui.cc
@@ -64,7 +64,7 @@
 #include "chrome/browser/policy/policy_domain_descriptor.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_set.h"
-#include "components/policy/core/common/policy_schema.h"
+#include "components/policy/core/common/schema.h"
 #include "extensions/common/manifest.h"
 #include "extensions/common/manifest_constants.h"
 #endif
@@ -579,12 +579,11 @@ void PolicyUIHandler::SendPolicyNames() const {
     if (schema != schema_map.end()) {
       // Get policy names from the extension's policy schema.
       // Store in a map, not an array, for faster lookup on JS side.
-      const policy::PolicySchemaMap* policies = schema->second->GetProperties();
-      policy::PolicySchemaMap::const_iterator it_policies;
-      for (it_policies = policies->begin(); it_policies != policies->end();
-           ++it_policies) {
-        policy_names->SetBoolean(it_policies->first, true);
-      }
+      policy::Schema policy_schema = schema->second;
+      policy::Schema::Iterator it_policies =
+          policy_schema.GetPropertiesIterator();
+      for (; !it_policies.IsAtEnd(); it_policies.Advance())

[bartfab (slow), 2013/09/24 10:03:55]

You don't use it_policies outside the for loop, why not constrain it to that
scope?

[Joao da Silva, 2013/09/24 10:16:31]

Done.

+        policy_names->SetBoolean(it_policies.key(), true);
     }
     extension_value->Set("policyNames", policy_names);
     extension_policy_names->Set(extension->id(), extension_value);