diff options
author | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-25 07:40:28 +0000 |
---|---|---|
committer | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-02-25 07:40:28 +0000 |
commit | 49fa9d59bca4ba0118a3e65f7cad5886cc4d5c3e (patch) | |
tree | e429cd276405e05073dee2a7f3a463001efa224f /net/base/host_resolver_impl.cc | |
parent | b19171c25e41bac6980550abed63df80165b5f1e (diff) | |
download | chromium_src-49fa9d59bca4ba0118a3e65f7cad5886cc4d5c3e.zip chromium_src-49fa9d59bca4ba0118a3e65f7cad5886cc4d5c3e.tar.gz chromium_src-49fa9d59bca4ba0118a3e65f7cad5886cc4d5c3e.tar.bz2 |
Revert 39996 - Refine IPv6 probe to require that the client has an IPv6 address on an interface
This currently only works on Posix, not windows.
Network changes are monitored, and the test is repeated each time interfaces
change (which is a subset of any IP addresses changing).
The test performed is still relatively low latency, and we *may* need
to eventually move to an high latency test, such as a DNS resolution,
or an actual test connection. If we move in that direction, then we'll
need to post a task to perform the work, rather than immediately returning.
BUG=25680
BUG=12754
r=wtc,eroman
Review URL: http://codereview.chromium.org/652072
TBR=jar@chromium.org
Review URL: http://codereview.chromium.org/660073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39998 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base/host_resolver_impl.cc')
-rw-r--r-- | net/base/host_resolver_impl.cc | 23 |
1 files changed, 1 insertions, 22 deletions
diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc index 2f50b10..184324b 100644 --- a/net/base/host_resolver_impl.cc +++ b/net/base/host_resolver_impl.cc @@ -20,7 +20,6 @@ #include "net/base/host_resolver_proc.h" #include "net/base/load_log.h" #include "net/base/net_errors.h" -#include "net/base/net_util.h" #include "net/base/network_change_notifier.h" #if defined(OS_WIN) @@ -576,8 +575,7 @@ HostResolverImpl::HostResolverImpl( resolver_proc_(resolver_proc), default_address_family_(ADDRESS_FAMILY_UNSPECIFIED), shutdown_(false), - network_change_notifier_(network_change_notifier), - ipv6_probe_monitoring_(false) { + network_change_notifier_(network_change_notifier) { DCHECK_GT(max_jobs, 0u); // It is cumbersome to expose all of the constraints in the constructor, @@ -738,17 +736,6 @@ void HostResolverImpl::RemoveObserver(HostResolver::Observer* observer) { observers_.erase(it); } -void HostResolverImpl::SetDefaultAddressFamily(AddressFamily address_family) { - ipv6_probe_monitoring_ = false; - default_address_family_ = address_family; -} - -void HostResolverImpl::ProbeIPv6Support() { - DCHECK(!ipv6_probe_monitoring_); - ipv6_probe_monitoring_ = true; - OnIPAddressChanged(); // Give initial setup call. -} - void HostResolverImpl::Shutdown() { shutdown_ = true; @@ -990,14 +977,6 @@ void HostResolverImpl::OnCancelRequest(LoadLog* load_log, void HostResolverImpl::OnIPAddressChanged() { if (cache_.get()) cache_->clear(); - if (ipv6_probe_monitoring_) { - bool support = IPv6Supported(); - default_address_family_ = support ? ADDRESS_FAMILY_UNSPECIFIED - : ADDRESS_FAMILY_IPV4; - LOG(INFO) << "IPv6Probe forced AddressFamily setting to " - << (support ? "ADDRESS_FAMILY_UNSPECIFIED" - : "ADDRESS_FAMILY_IPV4"); - } } // static |