Return both IPv6 and IPv4 address from the lookup.

webrtc/base/nethelpers.cc

Index: webrtc/base/nethelpers.cc
diff --git a/webrtc/base/nethelpers.cc b/webrtc/base/nethelpers.cc
index fddfdaf58b5832318fe4f73d4f2ad7262f5b03bf..aeb467c9ca100bcabaa36d532b5e939f5797a79f 100644
--- a/webrtc/base/nethelpers.cc
+++ b/webrtc/base/nethelpers.cc
@@ -44,8 +44,7 @@ int ResolveHostname(const std::string& hostname, int family,
   addresses->clear();
   struct addrinfo* result = NULL;
   struct addrinfo hints = {0};
-  // TODO(djw): For now this is IPv4 only so existing users remain unaffected.
-  hints.ai_family = AF_INET;
+  hints.ai_family = family;

[honghaiz3, 2016/06/24 01:19:08]

This will be essentially using AF_UNSPEC because the family coming from an IP
that is unresolved. 

[pthatcher1, 2016/06/24 18:36:05]

I think we should have a really big comment right here.  

Perhaps something like:


|family| here will almost always be AF_UNSPEC, because |family| comes from
AsyncResolver::addr_.family(), which is a SocketAddress, and a SocketAddress,
when constructed with a hostname is given a family of AF_UNSPEC.  However, if
someday in the future a way to construct a SocketAddress with both a hostname
and a family other than AF_UNSPEC, then it would be possible to get a specific
family value here.

The behavior of AF_UNSPEC is roughly "get both ipv4 and ipv6", as documented by
the various operating systems:

Linux: 
http://man7.org/linux/man-pages/man3/getaddrinfo.3.html
Windows: 
https://msdn.microsoft.com/en-us/library/windows/desktop/ms738520(v=vs.85).aspx
Mac:
https://developer.apple.com/legacy/library/documentation/Darwin/Reference/Man...
Android (source code, not documentation): 
https://android.googlesource.com/platform/bionic/+/7e0bfb511e85834d7c6cb96312...

[honghaiz3, 2016/06/24 21:26:39]

Done. Thanks!

   hints.ai_flags = AI_ADDRCONFIG;
   int ret = getaddrinfo(hostname.c_str(), NULL, &hints, &result);
   if (ret != 0) {