Push IPv6 addresses to libjingle PortAllocator.

content/renderer/p2p/ipc_network_manager.cc

 // Copyright (c) 2012 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 "content/renderer/p2p/ipc_network_manager.h"
 
 #include "base/bind.h"
 #include "base/sys_byteorder.h"
 #include "net/base/net_util.h"
 
@@ skipping 31 matching lines @@
 void IpcNetworkManager::OnNetworkListChanged(
     const net::NetworkInterfaceList& list) {
 
   // Update flag if network list received for the first time.
   if (!network_list_received_)
     network_list_received_ = true;
 
   std::vector<talk_base::Network*> networks;
   for (net::NetworkInterfaceList::const_iterator it = list.begin();
        it != list.end(); it++) {
-    uint32 address;
-    if (it->address.size() != net::kIPv4AddressSize)
-      continue;
-    memcpy(&address, &it->address[0], sizeof(uint32));
-    address = talk_base::NetworkToHost32(address);
-    talk_base::Network* network = new talk_base::Network(
-        it->name, it->name, talk_base::IPAddress(address), 32);
-    network->AddIP(talk_base::IPAddress(address));
-    networks.push_back(network);
+    if (it->address.size() == net::kIPv4AddressSize) {
+      uint32 address;
+      memcpy(&address, &it->address[0], sizeof(uint32));
+      address = talk_base::NetworkToHost32(address);
+      talk_base::Network* network = new talk_base::Network(
+          it->name, it->name, talk_base::IPAddress(address), 32);

[Ronghua Wu (Left Chromium), 2013/10/18 23:38:28]

Should we use NetworkInterface::network_prefix as the prefix_length?

[Mallinath (Gone from Chromium), 2013/10/19 00:09:39]

Now it's more aligned with the libjingle native code and meaning of prefix and
prefix length.

On 2013/10/18 23:38:28, Ronghua Wu wrote:

+      network->AddIP(talk_base::IPAddress(address));
+      networks.push_back(network);
+    } else if (it->address.size() == net::kIPv6AddressSize) {
+      in6_addr address;
+      memcpy(&address, &it->address[0], sizeof(in6_addr));
+      talk_base::IPAddress ip6_addr(address);
+      talk_base::Network* network = new talk_base::Network(
+          it->name, it->name, ip6_addr, 64);

[Ronghua Wu (Left Chromium), 2013/10/18 23:38:28]

dito

[Mallinath (Gone from Chromium), 2013/10/19 00:09:39]

same as above.
On 2013/10/18 23:38:28, Ronghua Wu wrote:

+      network->AddIP(ip6_addr);
+      networks.push_back(network);
+    }
   }
 
   bool changed = false;
   MergeNetworkList(networks, &changed);
   if (changed)
     SignalNetworksChanged();
 }
 
 void IpcNetworkManager::SendNetworksChangedSignal() {
   SignalNetworksChanged();
 }
 
 }  // namespace content