[net/dns] Removes locking from DnsConfigServiceWin and adds local computer name.

net/base/host_resolver_impl.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 "net/base/host_resolver_impl.h"
 
 #if defined(OS_WIN)
 #include <Winsock2.h>
 #elif defined(OS_POSIX)
 #include <netdb.h>
@@ skipping 1788 matching lines @@
     const RequestInfo& info) const {
   HostResolverFlags effective_flags =
       info.host_resolver_flags() | additional_resolver_flags_;
   AddressFamily effective_address_family = info.address_family();
   if (effective_address_family == ADDRESS_FAMILY_UNSPECIFIED &&
       default_address_family_ != ADDRESS_FAMILY_UNSPECIFIED) {
     effective_address_family = default_address_family_;
     if (ipv6_probe_monitoring_)
       effective_flags |= HOST_RESOLVER_DEFAULT_FAMILY_SET_DUE_TO_NO_IPV6;
   }
-  return Key(info.hostname(), effective_address_family, effective_flags);
+  return Key(StringToLowerASCII(info.hostname()),

[mmenke, 2012/03/22 19:36:08]

May want to have a unit test that depends on this.

[cbentzel, 2012/03/22 19:45:15]

I'm not sure you want to do this here. It looks like NetBIOS is case-sensitive
and this would get impacted.

[mmenke, 2012/03/22 19:55:09]

Err...wait...Doesn't GURL automatically lowercase URLs?  I try to visit
http://FLYING_GOAT/, and end up at http://flying_goat/

[mmenke, 2012/03/22 19:58:36]

Domains used by PAC script don't go through a GURL intermediary.  Not sure about
other cases.

+             effective_address_family,
+             effective_flags);
 }
 
 void HostResolverImpl::AbortAllInProgressJobs() {
   // In Abort, a Request callback could spawn new Jobs with matching keys, so
   // first collect and remove all running jobs from |jobs_|.
   std::vector<Job*> jobs_to_abort;
   for (JobMap::iterator it = jobs_.begin(); it != jobs_.end(); ) {
     Job* job = it->second;
     if (job->is_running()) {
       jobs_to_abort.push_back(job);
@@ skipping 90 matching lines @@
 
   if (self && dns_config.IsValid())
     TryServingAllJobsFromHosts();
 }
 
 bool HostResolverImpl::HaveDnsConfig() const {
   return (dns_client_.get() != NULL) && (dns_client_->GetConfig() != NULL);
 }
 
 }  // namespace net