Commit 4c60e323 authored by guoweis's avatar guoweis Committed by Commit bot

Rename network_prefix to prefix_length in NetworkInterface structure.

BUG=

Review URL: https://codereview.chromium.org/730623002

Cr-Commit-Position: refs/heads/master@{#305017}
parent cdc61efc
...@@ -32,7 +32,7 @@ void GetNetworkListOnFileThread( ...@@ -32,7 +32,7 @@ void GetNetworkListOnFileThread(
net::AddressFamily address_family = net::AddressFamily address_family =
net::GetAddressFamily(networks[i].address); net::GetAddressFamily(networks[i].address);
if (address_family == net::ADDRESS_FAMILY_IPV4 && if (address_family == net::ADDRESS_FAMILY_IPV4 &&
networks[i].network_prefix >= 24) { networks[i].prefix_length >= 24) {
ip4_networks.push_back(networks[i]); ip4_networks.push_back(networks[i]);
} }
} }
...@@ -141,7 +141,7 @@ int PrivetTrafficDetector::Bind() { ...@@ -141,7 +141,7 @@ int PrivetTrafficDetector::Bind() {
bool PrivetTrafficDetector::IsSourceAcceptable() const { bool PrivetTrafficDetector::IsSourceAcceptable() const {
for (size_t i = 0; i < networks_.size(); ++i) { for (size_t i = 0; i < networks_.size(); ++i) {
if (net::IPNumberMatchesPrefix(recv_addr_.address(), networks_[i].address, if (net::IPNumberMatchesPrefix(recv_addr_.address(), networks_[i].address,
networks_[i].network_prefix)) { networks_[i].prefix_length)) {
return true; return true;
} }
} }
......
...@@ -28,7 +28,7 @@ IPC_STRUCT_TRAITS_BEGIN(net::NetworkInterface) ...@@ -28,7 +28,7 @@ IPC_STRUCT_TRAITS_BEGIN(net::NetworkInterface)
IPC_STRUCT_TRAITS_MEMBER(name) IPC_STRUCT_TRAITS_MEMBER(name)
IPC_STRUCT_TRAITS_MEMBER(type) IPC_STRUCT_TRAITS_MEMBER(type)
IPC_STRUCT_TRAITS_MEMBER(address) IPC_STRUCT_TRAITS_MEMBER(address)
IPC_STRUCT_TRAITS_MEMBER(network_prefix) IPC_STRUCT_TRAITS_MEMBER(prefix_length)
IPC_STRUCT_TRAITS_MEMBER(ip_address_attributes) IPC_STRUCT_TRAITS_MEMBER(ip_address_attributes)
IPC_STRUCT_TRAITS_END() IPC_STRUCT_TRAITS_END()
......
...@@ -83,12 +83,9 @@ void IpcNetworkManager::OnNetworkListChanged( ...@@ -83,12 +83,9 @@ void IpcNetworkManager::OnNetworkListChanged(
memcpy(&address, &it->address[0], sizeof(uint32)); memcpy(&address, &it->address[0], sizeof(uint32));
address = rtc::NetworkToHost32(address); address = rtc::NetworkToHost32(address);
rtc::IPAddress prefix = rtc::IPAddress prefix =
rtc::TruncateIP(rtc::IPAddress(address), it->network_prefix); rtc::TruncateIP(rtc::IPAddress(address), it->prefix_length);
rtc::Network* network = rtc::Network* network =
new rtc::Network(it->name, new rtc::Network(it->name, it->name, prefix, it->prefix_length,
it->name,
prefix,
it->network_prefix,
ConvertConnectionTypeToAdapterType(it->type)); ConvertConnectionTypeToAdapterType(it->type));
network->AddIP(rtc::IPAddress(address)); network->AddIP(rtc::IPAddress(address));
networks.push_back(network); networks.push_back(network);
...@@ -99,12 +96,9 @@ void IpcNetworkManager::OnNetworkListChanged( ...@@ -99,12 +96,9 @@ void IpcNetworkManager::OnNetworkListChanged(
rtc::IPAddress ip6_addr(address); rtc::IPAddress ip6_addr(address);
if (!rtc::IPIsPrivate(ip6_addr)) { if (!rtc::IPIsPrivate(ip6_addr)) {
rtc::IPAddress prefix = rtc::IPAddress prefix =
rtc::TruncateIP(rtc::IPAddress(ip6_addr), it->network_prefix); rtc::TruncateIP(rtc::IPAddress(ip6_addr), it->prefix_length);
rtc::Network* network = rtc::Network* network =
new rtc::Network(it->name, new rtc::Network(it->name, it->name, prefix, it->prefix_length,
it->name,
prefix,
it->network_prefix,
ConvertConnectionTypeToAdapterType(it->type)); ConvertConnectionTypeToAdapterType(it->type));
network->AddIP(ip6_addr); network->AddIP(ip6_addr);
networks.push_back(network); networks.push_back(network);
......
...@@ -720,7 +720,7 @@ void SocketGetNetworkListFunction::SendResponseOnUIThread( ...@@ -720,7 +720,7 @@ void SocketGetNetworkListFunction::SendResponseOnUIThread(
make_linked_ptr(new core_api::socket::NetworkInterface); make_linked_ptr(new core_api::socket::NetworkInterface);
info->name = i->name; info->name = i->name;
info->address = net::IPAddressToString(i->address); info->address = net::IPAddressToString(i->address);
info->prefix_length = i->network_prefix; info->prefix_length = i->prefix_length;
create_arg.push_back(info); create_arg.push_back(info);
} }
......
...@@ -70,7 +70,7 @@ void SystemNetworkGetNetworkInterfacesFunction::SendResponseOnUIThread( ...@@ -70,7 +70,7 @@ void SystemNetworkGetNetworkInterfacesFunction::SendResponseOnUIThread(
make_linked_ptr(new core_api::system_network::NetworkInterface); make_linked_ptr(new core_api::system_network::NetworkInterface);
info->name = i->name; info->name = i->name;
info->address = net::IPAddressToString(i->address); info->address = net::IPAddressToString(i->address);
info->prefix_length = i->network_prefix; info->prefix_length = i->prefix_length;
create_arg.push_back(info); create_arg.push_back(info);
} }
......
...@@ -1015,8 +1015,7 @@ bool IsLocalhost(const std::string& host) { ...@@ -1015,8 +1015,7 @@ bool IsLocalhost(const std::string& host) {
} }
NetworkInterface::NetworkInterface() NetworkInterface::NetworkInterface()
: type(NetworkChangeNotifier::CONNECTION_UNKNOWN), : type(NetworkChangeNotifier::CONNECTION_UNKNOWN), prefix_length(0) {
network_prefix(0) {
} }
NetworkInterface::NetworkInterface(const std::string& name, NetworkInterface::NetworkInterface(const std::string& name,
...@@ -1024,14 +1023,14 @@ NetworkInterface::NetworkInterface(const std::string& name, ...@@ -1024,14 +1023,14 @@ NetworkInterface::NetworkInterface(const std::string& name,
uint32 interface_index, uint32 interface_index,
NetworkChangeNotifier::ConnectionType type, NetworkChangeNotifier::ConnectionType type,
const IPAddressNumber& address, const IPAddressNumber& address,
uint32 network_prefix, uint32 prefix_length,
int ip_address_attributes) int ip_address_attributes)
: name(name), : name(name),
friendly_name(friendly_name), friendly_name(friendly_name),
interface_index(interface_index), interface_index(interface_index),
type(type), type(type),
address(address), address(address),
network_prefix(network_prefix), prefix_length(prefix_length),
ip_address_attributes(ip_address_attributes) { ip_address_attributes(ip_address_attributes) {
} }
......
...@@ -468,7 +468,7 @@ struct NET_EXPORT NetworkInterface { ...@@ -468,7 +468,7 @@ struct NET_EXPORT NetworkInterface {
uint32 interface_index, uint32 interface_index,
NetworkChangeNotifier::ConnectionType type, NetworkChangeNotifier::ConnectionType type,
const IPAddressNumber& address, const IPAddressNumber& address,
uint32 network_prefix, uint32 prefix_length,
int ip_address_attributes); int ip_address_attributes);
~NetworkInterface(); ~NetworkInterface();
...@@ -477,7 +477,7 @@ struct NET_EXPORT NetworkInterface { ...@@ -477,7 +477,7 @@ struct NET_EXPORT NetworkInterface {
uint32 interface_index; // Always 0 on Android. uint32 interface_index; // Always 0 on Android.
NetworkChangeNotifier::ConnectionType type; NetworkChangeNotifier::ConnectionType type;
IPAddressNumber address; IPAddressNumber address;
uint32 network_prefix; uint32 prefix_length;
int ip_address_attributes; // Combination of |IPAddressAttributes|. int ip_address_attributes; // Combination of |IPAddressAttributes|.
}; };
......
...@@ -831,8 +831,8 @@ TEST(NetUtilTest, GetNetworkList) { ...@@ -831,8 +831,8 @@ TEST(NetUtilTest, GetNetworkList) {
} }
} }
EXPECT_FALSE(all_zeroes); EXPECT_FALSE(all_zeroes);
EXPECT_GT(it->network_prefix, 1u); EXPECT_GT(it->prefix_length, 1u);
EXPECT_LE(it->network_prefix, it->address.size() * 8); EXPECT_LE(it->prefix_length, it->address.size() * 8);
#if defined(OS_WIN) #if defined(OS_WIN)
// On Windows |name| is NET_LUID. // On Windows |name| is NET_LUID.
...@@ -975,7 +975,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -975,7 +975,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
GetInterfaceNameVM)); GetInterfaceNameVM));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_vm); EXPECT_EQ(results[0].name, ifname_vm);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
results.clear(); results.clear();
...@@ -1017,7 +1017,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1017,7 +1017,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
GetInterfaceName)); GetInterfaceName));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_em1); EXPECT_EQ(results[0].name, ifname_em1);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_TEMPORARY); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_TEMPORARY);
results.clear(); results.clear();
...@@ -1035,7 +1035,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1035,7 +1035,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
GetInterfaceName)); GetInterfaceName));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_em1); EXPECT_EQ(results[0].name, ifname_em1);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_DEPRECATED); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_DEPRECATED);
results.clear(); results.clear();
...@@ -1079,7 +1079,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1079,7 +1079,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
&ip_attributes_getter)); &ip_attributes_getter));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_vm); EXPECT_EQ(results[0].name, ifname_vm);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
results.clear(); results.clear();
...@@ -1113,7 +1113,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1113,7 +1113,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
&ip_attributes_getter)); &ip_attributes_getter));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_em1); EXPECT_EQ(results[0].name, ifname_em1);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_TEMPORARY); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_TEMPORARY);
results.clear(); results.clear();
...@@ -1128,7 +1128,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1128,7 +1128,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
&ip_attributes_getter)); &ip_attributes_getter));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_em1); EXPECT_EQ(results[0].name, ifname_em1);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_DEPRECATED); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_DEPRECATED);
results.clear(); results.clear();
...@@ -1237,7 +1237,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1237,7 +1237,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
&results, INCLUDE_HOST_SCOPE_VIRTUAL_INTERFACES, true, &adapter_address)); &results, INCLUDE_HOST_SCOPE_VIRTUAL_INTERFACES, true, &adapter_address));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_vm); EXPECT_EQ(results[0].name, ifname_vm);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_NONE); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_NONE);
results.clear(); results.clear();
...@@ -1279,7 +1279,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1279,7 +1279,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
&results, INCLUDE_HOST_SCOPE_VIRTUAL_INTERFACES, true, &adapter_address)); &results, INCLUDE_HOST_SCOPE_VIRTUAL_INTERFACES, true, &adapter_address));
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].name, ifname_em1); EXPECT_EQ(results[0].name, ifname_em1);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_TEMPORARY); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_TEMPORARY);
results.clear(); results.clear();
...@@ -1298,7 +1298,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) { ...@@ -1298,7 +1298,7 @@ TEST(NetUtilTest, GetNetworkListTrimming) {
EXPECT_EQ(results.size(), 1ul); EXPECT_EQ(results.size(), 1ul);
EXPECT_EQ(results[0].friendly_name, "FriendlyInterfaceName"); EXPECT_EQ(results[0].friendly_name, "FriendlyInterfaceName");
EXPECT_EQ(results[0].name, ifname_em1); EXPECT_EQ(results[0].name, ifname_em1);
EXPECT_EQ(results[0].network_prefix, 1ul); EXPECT_EQ(results[0].prefix_length, 1ul);
EXPECT_EQ(results[0].address, ipv6_address); EXPECT_EQ(results[0].address, ipv6_address);
EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_DEPRECATED); EXPECT_EQ(results[0].ip_address_attributes, IP_ADDRESS_ATTRIBUTE_DEPRECATED);
results.clear(); results.clear();
......
...@@ -117,7 +117,7 @@ bool GetNetworkListImpl(NetworkInterfaceList* networks, ...@@ -117,7 +117,7 @@ bool GetNetworkListImpl(NetworkInterfaceList* networks,
if (endpoint.FromSockAddr(address->Address.lpSockaddr, if (endpoint.FromSockAddr(address->Address.lpSockaddr,
address->Address.iSockaddrLength)) { address->Address.iSockaddrLength)) {
// XP has no OnLinkPrefixLength field. // XP has no OnLinkPrefixLength field.
size_t net_prefix = is_xp ? 0 : address->OnLinkPrefixLength; size_t prefix_length = is_xp ? 0 : address->OnLinkPrefixLength;
if (is_xp) { if (is_xp) {
// Prior to Windows Vista the FirstPrefix pointed to the list with // Prior to Windows Vista the FirstPrefix pointed to the list with
// single prefix for each IP address assigned to the adapter. // single prefix for each IP address assigned to the adapter.
...@@ -133,7 +133,8 @@ bool GetNetworkListImpl(NetworkInterfaceList* networks, ...@@ -133,7 +133,8 @@ bool GetNetworkListImpl(NetworkInterfaceList* networks,
IPNumberMatchesPrefix(endpoint.address(), IPNumberMatchesPrefix(endpoint.address(),
network_endpoint.address(), network_endpoint.address(),
prefix->PrefixLength)) { prefix->PrefixLength)) {
net_prefix = std::max<size_t>(net_prefix, prefix->PrefixLength); prefix_length =
std::max<size_t>(prefix_length, prefix->PrefixLength);
} }
} }
} }
...@@ -165,7 +166,7 @@ bool GetNetworkListImpl(NetworkInterfaceList* networks, ...@@ -165,7 +166,7 @@ bool GetNetworkListImpl(NetworkInterfaceList* networks,
adapter->AdapterName, adapter->AdapterName,
base::SysWideToNativeMB(adapter->FriendlyName), index, base::SysWideToNativeMB(adapter->FriendlyName), index,
GetNetworkInterfaceType(adapter->IfType), endpoint.address(), GetNetworkInterfaceType(adapter->IfType), endpoint.address(),
net_prefix, ip_address_attributes)); prefix_length, ip_address_attributes));
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment