commit e49d7710a951d161bf434de84e3dd34f68f47a26
author Vitaly Buka <vitalybuka@chromium.org> Thu Aug 06 01:35:11 2015 -0700
committer Alex Vakulenko <avakulenko@google.com> Thu Aug 06 14:23:21 2015 -0700
tree 085730f14f94858c0570f291e8d585fd2d297f1e
parent 0b4e1fd405dde857af81711195c7b0b0bcc5e6b1

libweave: Remove include of chromeos/map_utils.h

It was used only to find value in std::map.
Also replaces EnumToString was used for mapping.

BUG=brillo:1257
TEST='FEATURES=test emerge-gizmo libweave buffet'

Change-Id: I5eb499a9a14c4b6c769cf4925ccfd4d8f3d6e99b

libweave/src/commands/object_schema.cc

diff --git a/libweave/src/commands/object_schema.cc b/libweave/src/commands/object_schema.cc
index 1aa3bfa..b08d143 100644
--- a/libweave/src/commands/object_schema.cc
+++ b/libweave/src/commands/object_schema.cc
@@ -9,7 +9,7 @@
 
 #include <base/logging.h>
 #include <base/values.h>
-#include <chromeos/map_utils.h>
+#include <weave/enum_to_string.h>
 
 #include "libweave/src/commands/prop_types.h"
 #include "libweave/src/commands/prop_values.h"
@@ -260,8 +260,23 @@
   return prop;
 }
 
+const EnumToStringMap<base::Value::Type>::Map kMap[] = {
+    {base::Value::TYPE_NULL, "Null"},
+    {base::Value::TYPE_BOOLEAN, "Boolean"},
+    {base::Value::TYPE_INTEGER, "Integer"},
+    {base::Value::TYPE_DOUBLE, "Double"},
+    {base::Value::TYPE_STRING, "String"},
+    {base::Value::TYPE_BINARY, "Binary"},
+    {base::Value::TYPE_DICTIONARY, "Object"},
+    {base::Value::TYPE_LIST, "Array"},
+};
+
 }  // anonymous namespace
 
+template <>
+EnumToStringMap<base::Value::Type>::EnumToStringMap()
+    : EnumToStringMap(kMap) {}
+
 ObjectSchema::ObjectSchema() {
 }
 ObjectSchema::~ObjectSchema() {
@@ -352,21 +367,10 @@
     // Full parameter definition.
     return PropFromJsonObject(value, base_schema, error);
   }
-  static const std::map<base::Value::Type, const char*> type_names = {
-      {base::Value::TYPE_NULL, "Null"},
-      {base::Value::TYPE_BOOLEAN, "Boolean"},
-      {base::Value::TYPE_INTEGER, "Integer"},
-      {base::Value::TYPE_DOUBLE, "Double"},
-      {base::Value::TYPE_STRING, "String"},
-      {base::Value::TYPE_BINARY, "Binary"},
-      {base::Value::TYPE_DICTIONARY, "Object"},
-      {base::Value::TYPE_LIST, "Array"},
-  };
-  const char* type_name =
-      chromeos::GetOrDefault(type_names, value.GetType(), "<unknown>");
   chromeos::Error::AddToPrintf(error, FROM_HERE, errors::commands::kDomain,
                                errors::commands::kUnknownType,
-                               "Unexpected JSON value type: %s", type_name);
+                               "Unexpected JSON value type: %s",
+                               EnumToString(value.GetType()).c_str());
   return nullptr;
 }