Resolve addresses of STUN/Relay servers in JingleInfoRequest.

remoting/jingle_glue/host_address_resolver.h

Index: remoting/jingle_glue/host_address_resolver.h
diff --git a/remoting/jingle_glue/host_address_resolver.h b/remoting/jingle_glue/host_address_resolver.h
new file mode 100644
index 0000000000000000000000000000000000000000..1cece0b7ba4d3a6e05595a7e56c3a37dbc7518f9
--- /dev/null
+++ b/remoting/jingle_glue/host_address_resolver.h
@@ -0,0 +1,46 @@
+// 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 REMOTING_JINGLE_GLUE_HOST_ADDRESS_RESOLVER_H_
+#define REMOTING_JINGLE_GLUE_HOST_ADDRESS_RESOLVER_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "third_party/libjingle/source/talk/base/sigslot.h"
+#include "third_party/libjingle/source/talk/base/socketaddress.h"
+
+namespace remoting {
+
+// TODO(sergeyu): Move HostAddressRequest and HostAddressResolver to
+// libjingle and use them in StunPort.
+
+class HostAddressRequest {

[Wez, 2011/08/08 23:07:59]

Isn't this just a HostResolver?

[Sergey Ulanov, 2011/08/09 01:42:16]

Done.

+ public:
+  HostAddressRequest() { }
+  virtual ~HostAddressRequest() { }
+
+  virtual void Request(const std::string& name) = 0;
+
+  sigslot::signal2<HostAddressRequest*,
+                   const talk_base::SocketAddress&> SignalDone;

[Wez, 2011/08/08 23:07:59]

This only supports a single resolved address, whereas a host may resolve to many
addresses, as well as to alternate names.  Is it a problem that this interface
only supports IPv4?

[Sergey Ulanov, 2011/08/09 01:42:16]

It reflects the current behavior implemented in libjingle. Normally names of
STUN servers resolve to single address, and I'm not sure will will ever need
multiple addresses for the same domain name. The server may always provide
multiple stun server names if it is necessary to have multiple STUN servers.
It will support IPv6 when IPv6 support is implemented in
talk_base::SocketAddress.

+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(HostAddressRequest);
+};
+
+class HostAddressResolver {

[Wez, 2011/08/08 23:07:59]

Isn't this a HostResolverFactory?

[Sergey Ulanov, 2011/08/09 01:42:16]

Done.

+ public:
+  HostAddressResolver() { }
+  virtual ~HostAddressResolver() { }
+
+  virtual HostAddressRequest* CreateRequest() = 0;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(HostAddressResolver);
+};
+
+}  // namespace remoting
+
+#endif  // REMOTING_JINGLE_GLUE_HOST_ADDRESS_RESOLVER_H_