ConfigurationPolicyPrefStore refactoring to surface error messages.

chrome/browser/policy/policy_error_map.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_POLICY_ERROR_MAP_H_
+#define CHROME_BROWSER_POLICY_POLICY_ERROR_MAP_H_
+#pragma once
+
+#include <map>
+
+#include "base/values.h"
+#include "policy/configuration_policy_type.h"
+
+namespace policy {
+
+// Wrapper class around a std::multimap<ConfigurationPolicyType, StringValue*>
+// used to associate policies with error messages generated when a policy fails
+// to be applied.
+class PolicyErrorMap {
+ public:
+  typedef std::multimap<ConfigurationPolicyType, StringValue*> PolicyMapType;

[Mattias Nissler (ping if slow), 2011/09/20 13:12:25]

Why is this a StringValue* instead of a plain string?

[simo, 2011/09/22 11:43:26]

For some reason I was thinking I needed to put StringValues in to return them in
ListValue but obviously that's not the case. I'll change it to string. 

+  typedef PolicyMapType::const_iterator const_iterator;
+
+  PolicyErrorMap();
+  virtual ~PolicyErrorMap();
+
+  // Takes ownership of |error_message|.
+  void AddError(ConfigurationPolicyType policy, StringValue* error_message);
+
+  // Returns a list of all the error messages stored for |policy|. The caller
+  // acquires ownership of the returned ListValue pointer.
+  ListValue* GetErrors(ConfigurationPolicyType policy) const;
+
+  bool empty() const;
+  size_t size() const;
+
+  const_iterator begin() const;
+  const_iterator end() const;
+
+  void Clear();
+
+ private:
+  PolicyMapType map_;
+
+  DISALLOW_COPY_AND_ASSIGN(PolicyErrorMap);

[Mattias Nissler (ping if slow), 2011/09/20 13:12:25]

#include "base/basic_types.h"

[simo, 2011/09/22 11:43:26]

I couldn't find "base/basic_types.h" but I included "base/basictypes.h"

+};
+
+}  // namespace policy
+
+#endif  // CHROME_BROWSER_POLICY_POLICY_ERROR_MAP_H_