summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormartijn <martijn@martijnc.be>2016-03-17 03:09:09 -0700
committerCommit bot <commit-bot@chromium.org>2016-03-17 10:10:05 +0000
commitea3920faf5dfd8e3f50a99085bcf020978fb7371 (patch)
tree3a1bd2424e2ce550ebbe4ebab87ad80e065c3d6d
parentc69f2f97d6b9e4cf13d8fad44fad45eded6a119f (diff)
downloadchromium_src-ea3920faf5dfd8e3f50a99085bcf020978fb7371.zip
chromium_src-ea3920faf5dfd8e3f50a99085bcf020978fb7371.tar.gz
chromium_src-ea3920faf5dfd8e3f50a99085bcf020978fb7371.tar.bz2
Migrate chrome/browser/net/* to net::IPAddress.
BUG=496258 Review URL: https://codereview.chromium.org/1807663002 Cr-Commit-Position: refs/heads/master@{#381679}
-rw-r--r--chrome/browser/net/dns_probe_runner.cc1
-rw-r--r--chrome/browser/net/dns_probe_service.cc21
-rw-r--r--chrome/browser/net/dns_probe_test_util.cc6
3 files changed, 11 insertions, 17 deletions
diff --git a/chrome/browser/net/dns_probe_runner.cc b/chrome/browser/net/dns_probe_runner.cc
index fcd0d26..65bea52 100644
--- a/chrome/browser/net/dns_probe_runner.cc
+++ b/chrome/browser/net/dns_probe_runner.cc
@@ -26,7 +26,6 @@ using net::DnsClient;
using net::DnsResponse;
using net::DnsTransaction;
using net::DnsTransactionFactory;
-using net::IPAddressNumber;
using net::IPEndPoint;
using net::NetLog;
using net::NetworkChangeNotifier;
diff --git a/chrome/browser/net/dns_probe_service.cc b/chrome/browser/net/dns_probe_service.cc
index fcec9981..a16b65c5 100644
--- a/chrome/browser/net/dns_probe_service.cc
+++ b/chrome/browser/net/dns_probe_service.cc
@@ -4,11 +4,14 @@
#include "chrome/browser/net/dns_probe_service.h"
+#include <stdint.h>
+
#include <utility>
#include "base/metrics/field_trial.h"
#include "base/metrics/histogram.h"
#include "base/strings/string_number_conversions.h"
+#include "net/base/ip_address.h"
#include "net/base/ip_endpoint.h"
#include "net/dns/dns_client.h"
#include "net/dns/dns_config_service.h"
@@ -19,7 +22,6 @@ using base::StringToInt;
using error_page::DnsProbeStatus;
using net::DnsClient;
using net::DnsConfig;
-using net::IPAddressNumber;
using net::ParseIPLiteralToNumber;
using net::NetworkChangeNotifier;
@@ -34,15 +36,8 @@ const int kMaxResultAgeMs = 5000;
// The public DNS servers used by the DnsProbeService to verify internet
// connectivity.
-const char kGooglePublicDns1[] = "8.8.8.8";
-const char kGooglePublicDns2[] = "8.8.4.4";
-
-net::IPEndPoint MakeDnsEndPoint(const std::string& dns_ip_literal) {
- IPAddressNumber dns_ip_number;
- bool rv = ParseIPLiteralToNumber(dns_ip_literal, &dns_ip_number);
- DCHECK(rv);
- return net::IPEndPoint(dns_ip_number, net::dns_protocol::kDefaultPort);
-}
+const uint8_t kGooglePublicDns1[] = {8, 8, 8, 8};
+const uint8_t kGooglePublicDns2[] = {8, 8, 4, 4};
DnsProbeStatus EvaluateResults(DnsProbeRunner::Result system_result,
DnsProbeRunner::Result public_result) {
@@ -154,8 +149,10 @@ void DnsProbeService::SetSystemClientToCurrentConfig() {
void DnsProbeService::SetPublicClientToGooglePublicDns() {
DnsConfig public_config;
- public_config.nameservers.push_back(MakeDnsEndPoint(kGooglePublicDns1));
- public_config.nameservers.push_back(MakeDnsEndPoint(kGooglePublicDns2));
+ public_config.nameservers.push_back(net::IPEndPoint(
+ net::IPAddress(kGooglePublicDns1), net::dns_protocol::kDefaultPort));
+ public_config.nameservers.push_back(net::IPEndPoint(
+ net::IPAddress(kGooglePublicDns2), net::dns_protocol::kDefaultPort));
public_config.attempts = 1;
public_config.randomize_ports = false;
diff --git a/chrome/browser/net/dns_probe_test_util.cc b/chrome/browser/net/dns_probe_test_util.cc
index 5d51a32..60aa78c 100644
--- a/chrome/browser/net/dns_probe_test_util.cc
+++ b/chrome/browser/net/dns_probe_test_util.cc
@@ -7,23 +7,21 @@
#include <stdint.h>
#include "chrome/browser/net/dns_probe_runner.h"
+#include "net/base/ip_address.h"
#include "net/dns/dns_config_service.h"
#include "net/dns/dns_protocol.h"
using net::DnsClient;
using net::DnsConfig;
-using net::IPAddressNumber;
using net::MockDnsClientRule;
using net::MockDnsClientRuleList;
-using net::ParseIPLiteralToNumber;
namespace chrome_browser_net {
scoped_ptr<DnsClient> CreateMockDnsClientForProbes(
MockDnsClientRule::Result result) {
DnsConfig config;
- IPAddressNumber dns_ip;
- ParseIPLiteralToNumber("192.168.1.1", &dns_ip);
+ net::IPAddress dns_ip(192, 168, 1, 1);
const uint16_t kDnsPort = net::dns_protocol::kDefaultPort;
config.nameservers.push_back(net::IPEndPoint(dns_ip, kDnsPort));