[net/dns] Perform A/AAAA queries for AF_UNSPEC resolutions in parallel.

net/dns/host_resolver_impl.h

 // 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.
 
 #ifndef NET_DNS_HOST_RESOLVER_IMPL_H_
 #define NET_DNS_HOST_RESOLVER_IMPL_H_
 
 #include <map>
 
 #include "base/basictypes.h"
@@ skipping 205 matching lines @@
   // NetworkChangeNotifier::DNSObserver:
   virtual void OnDNSChanged() OVERRIDE;
 
   // True if have a DnsClient with a valid DnsConfig.
   bool HaveDnsConfig() const;
 
   // Called when a host name is successfully resolved and DnsTask was run on it
   // and resulted in |net_error|.
   void OnDnsTaskResolve(int net_error);
 
-  // Allows the tests to catch slots leaking out of the dispatcher.
-  size_t num_running_jobs_for_tests() const {
+  // Allows the tests to catch slots leaking out of the dispatcher.  One
+  // HostResolverImpl::Job moccupy multiple PrioritizedDispatcher job slots.

[szym, 2013/08/06 20:35:31]

moccupy => could occupy
Probably OK to skip HostResolverImpl::

[mmenke, 2013/08/19 17:31:11]

Done.
Since Job is overloaded (PrioritiedDispatcher has them and HostResolverImpl has
them), I prefer to keep the "HostResolverImpl::" prefix, to reduce ambiguity.

+  size_t num_running_dispatcher_jobs_for_tests() const {
     return dispatcher_.num_running_jobs();
   }
 
   // Cache of host resolution results.
   scoped_ptr<HostCache> cache_;
 
   // Map from HostCache::Key to a Job.
   JobMap jobs_;
 
   // Starts Jobs according to their priority and the configured limits.
@@ skipping 37 matching lines @@
 
   // Allow fallback to ProcTask if DnsTask fails.
   bool fallback_to_proctask_;
 
   DISALLOW_COPY_AND_ASSIGN(HostResolverImpl);
 };
 
 }  // namespace net
 
 #endif  // NET_DNS_HOST_RESOLVER_IMPL_H_