ConfigurationPolicyPrefStore refactoring to surface error messages.

chrome/browser/policy/configuration_policy_handler_list.h

+// Copyright (c) 2011 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_CONFIGURATION_POLICY_HANDLER_LIST_H_
+#define CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_HANDLER_LIST_H_
+#pragma once
+
+#include <vector>
+
+#include "base/values.h"
+#include "policy/configuration_policy_type.h"
+
+namespace policy {
+
+class ConfigurationPolicyHandlerInterface;
+
+typedef std::vector<ConfigurationPolicyHandlerInterface*> HandlerList;
+
+// Returns a pointer to a |HandlerList| containing all available handlers
+// derived from ConfigurationPolicyHandlerInterface.

[Mattias Nissler (ping if slow), 2011/09/26 13:30:48]

What we usually do in this case is passing in a vector pointer as parameter that
is then filled in by the function.

[simo, 2011/09/29 09:33:08]

Done.

+HandlerList* MakeConfigurationPolicyHandlers();
+
+// Lists policies that map to a single preference. These are handled by
+// SimplePolicyHandler. Each one of these policies has an entry in
+// kSimplePolicyMap with the following type.
+struct PolicyToPreferenceMapEntry {

[Mattias Nissler (ping if slow), 2011/09/26 13:30:48]

Do we actually need to publish this type? Can we just move it to the .cc file?

[simo, 2011/09/29 09:33:08]

I'm currently using it for the DefaultSearchPolicyHandler to list all default
search policies.

[Mattias Nissler (ping if slow), 2011/09/30 09:01:33]

Done.

+  base::Value::Type value_type;
+  ConfigurationPolicyType policy_type;
+  const char* preference_path;  // A DictionaryValue path, not a file path.
+};
+
+extern const PolicyToPreferenceMapEntry kSimplePolicyMap[];
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_HANDLER_LIST_H_