Add base::Value::GetTypeName().

components/policy/core/browser/configuration_policy_handler.cc

Index: components/policy/core/browser/configuration_policy_handler.cc
diff --git a/components/policy/core/browser/configuration_policy_handler.cc b/components/policy/core/browser/configuration_policy_handler.cc
index c7b3afa5b599f7b5424868fbe54ecfe153463699..c5d4bed0f39cb081e588faf1bf639c3f412724e7 100644
--- a/components/policy/core/browser/configuration_policy_handler.cc
+++ b/components/policy/core/browser/configuration_policy_handler.cc
@@ -30,18 +30,7 @@ namespace policy {
 // static
 std::string ConfigurationPolicyHandler::ValueTypeToString(

[pneubeck (no reviews), 2016/07/14 18:51:37]

maybe inline and remove this function?

[Lei Zhang, 2016/07/15 01:27:18]

It has a lot of callers. Added a TODO to make another pass.

     base::Value::Type type) {
-  static const char* strings[] = {
-    "null",
-    "boolean",
-    "integer",
-    "double",
-    "string",
-    "binary",
-    "dictionary",
-    "list"
-  };
-  CHECK(static_cast<size_t>(type) < arraysize(strings));
-  return std::string(strings[type]);
+  return std::string(base::Value::GetTypeName(type));
 }
 
 ConfigurationPolicyHandler::ConfigurationPolicyHandler() {