Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(169)

Unified Diff: net/dns/host_resolver_impl.cc

Issue 2831383003: Delete "Net.UnspecResolvedIPv6" histogram (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/dns/host_resolver_impl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/dns/host_resolver_impl.cc
diff --git a/net/dns/host_resolver_impl.cc b/net/dns/host_resolver_impl.cc
index fd4c065b7d7e724f47f3646ffd9fe32689b1a2e7..abfddf9be9d1cde596368117131fa8337e56e988 100644
--- a/net/dns/host_resolver_impl.cc
+++ b/net/dns/host_resolver_impl.cc
@@ -1635,22 +1635,6 @@ class HostResolverImpl::Job : public PrioritizedDispatcher::Job,
const AddressList& addr_list) {
DCHECK(is_proc_running());
- if (!resolver_->resolved_known_ipv6_hostname_ &&
- net_error == OK &&
- key_.address_family == ADDRESS_FAMILY_UNSPECIFIED) {
- if (key_.hostname == "www.google.com") {
- resolver_->resolved_known_ipv6_hostname_ = true;
- bool got_ipv6_address = false;
- for (size_t i = 0; i < addr_list.size(); ++i) {
- if (addr_list[i].GetFamily() == ADDRESS_FAMILY_IPV6) {
- got_ipv6_address = true;
- break;
- }
- }
- UMA_HISTOGRAM_BOOLEAN("Net.UnspecResolvedIPv6", got_ipv6_address);
- }
- }
-
if (dns_task_error_ != OK) {
base::TimeDelta duration = base::TimeTicks::Now() - start_time;
if (net_error == OK) {
@@ -2049,7 +2033,6 @@ HostResolverImpl::HostResolverImpl(
assume_ipv6_failure_on_wifi_(false),
use_local_ipv6_(false),
last_ipv6_probe_result_(true),
- resolved_known_ipv6_hostname_(false),
additional_resolver_flags_(0),
fallback_to_proctask_(true),
worker_task_runner_(std::move(worker_task_runner)),
@@ -2530,7 +2513,6 @@ void HostResolverImpl::TryServingAllJobsFromHosts() {
}
void HostResolverImpl::OnIPAddressChanged() {
- resolved_known_ipv6_hostname_ = false;
last_ipv6_probe_time_ = base::TimeTicks();
// Abandon all ProbeJobs.
probe_weak_ptr_factory_.InvalidateWeakPtrs();
« no previous file with comments | « net/dns/host_resolver_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698