Add remoteHost to Socket.

runtime/bin/socket.cc

Index: runtime/bin/socket.cc
diff --git a/runtime/bin/socket.cc b/runtime/bin/socket.cc
index 94cb46614a4377aee00fc4fb659811eaf9346558..6ec91037f24311582fa1dd86c6525185d3770065 100644
--- a/runtime/bin/socket.cc
+++ b/runtime/bin/socket.cc
@@ -153,15 +153,20 @@ void FUNCTION_NAME(Socket_GetPort)(Dart_NativeArguments args) {
 }
 
 
-void FUNCTION_NAME(Socket_GetRemotePort)(Dart_NativeArguments args) {
+void FUNCTION_NAME(Socket_GetRemotePeer)(Dart_NativeArguments args) {
   Dart_EnterScope();
   intptr_t socket =
       DartUtils::GetIntegerField(Dart_GetNativeArgument(args, 0),
                                  DartUtils::kIdFieldName);
   OSError os_error;
-  intptr_t port = Socket::GetRemotePort(socket);
-  if (port > 0) {
-    Dart_SetReturnValue(args, Dart_NewInteger(port));
+  intptr_t port = 0;
+  const char *host = NULL;

[Søren Gjesse, 2012/04/10 11:32:41]

How about

char host[INET_ADDRSTRLEN];

here? Then you don't need the allocation/deallocation.

Of cause you need to define a platform independent INET_ADDRSTRLEN here. Could
be named DART_INET_ADDRSTRLEN and you can assert that DART_INET_ADDRSTRLEN >=
INET_ADDRSTRLEN in all platform specific files.

+  if (Socket::GetRemotePeer(socket, &host, &port)) {
+    Dart_Handle list = Dart_NewList(2);
+    Dart_ListSetAt(list, 0, Dart_NewString(host));
+    Dart_ListSetAt(list, 1, Dart_NewInteger(port));
+    Dart_SetReturnValue(args, list);
+    delete[] host;
   } else {
     Dart_SetReturnValue(args, DartUtils::NewDartOSError());
   }