[Chromoting] Use google:remoting namespace to export remoting specific error codes

remoting/protocol/errors.cc

 // Copyright 2015 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.
 
 #include "remoting/protocol/errors.h"
 
 #include "base/logging.h"
 
 namespace remoting {
 namespace protocol {
 
+#define MAP_STRING_LITERAL(x) \

[Sergey Ulanov, 2016/06/02 09:26:03]

We don't use define like this for NameMapElement definitions elsewhere, so I'd
prefer to avoid it here as well.

[Hzj_jie, 2016/06/02 22:00:18]

Done.

+  { x, #x }
+const NameMapElement<ErrorCode> kErrorCodes[] = {

[Sergey Ulanov, 2016/06/02 09:26:03]

nit: kErrorCodeNames

[Hzj_jie, 2016/06/02 22:00:18]

Done.

+  MAP_STRING_LITERAL(OK),
+  MAP_STRING_LITERAL(PEER_IS_OFFLINE),
+  MAP_STRING_LITERAL(SESSION_REJECTED),
+  MAP_STRING_LITERAL(INCOMPATIBLE_PROTOCOL),
+  MAP_STRING_LITERAL(AUTHENTICATION_FAILED),
+  MAP_STRING_LITERAL(INVALID_ACCOUNT),
+  MAP_STRING_LITERAL(CHANNEL_CONNECTION_ERROR),
+  MAP_STRING_LITERAL(SIGNALING_ERROR),
+  MAP_STRING_LITERAL(SIGNALING_TIMEOUT),
+  MAP_STRING_LITERAL(HOST_OVERLOAD),
+  MAP_STRING_LITERAL(MAX_SESSION_LENGTH),
+  MAP_STRING_LITERAL(HOST_CONFIGURATION_ERROR),
+  MAP_STRING_LITERAL(UNKNOWN_ERROR),
+};
+
 #define RETURN_STRING_LITERAL(x) \

[Sergey Ulanov, 2016/06/02 09:26:03]

This define can be removed.

[Hzj_jie, 2016/06/02 22:00:18]

Done.

 case x: \
 return #x;
 
 const char* ErrorCodeToString(ErrorCode error) {
-  switch (error) {
-    RETURN_STRING_LITERAL(OK);
-    RETURN_STRING_LITERAL(PEER_IS_OFFLINE);
-    RETURN_STRING_LITERAL(SESSION_REJECTED);
-    RETURN_STRING_LITERAL(INCOMPATIBLE_PROTOCOL);
-    RETURN_STRING_LITERAL(AUTHENTICATION_FAILED);
-    RETURN_STRING_LITERAL(INVALID_ACCOUNT);
-    RETURN_STRING_LITERAL(CHANNEL_CONNECTION_ERROR);
-    RETURN_STRING_LITERAL(SIGNALING_ERROR);
-    RETURN_STRING_LITERAL(SIGNALING_TIMEOUT);
-    RETURN_STRING_LITERAL(HOST_OVERLOAD);
-    RETURN_STRING_LITERAL(MAX_SESSION_LENGTH);
-    RETURN_STRING_LITERAL(HOST_CONFIGURATION_ERROR);
-    RETURN_STRING_LITERAL(UNKNOWN_ERROR);
-  }
-  NOTREACHED();
-  return nullptr;
+  return ValueToName(kErrorCodes, error);
 }
 
 }  // namespace protocol
 }  // namespace remoting