Third patch in getting rid of caching MessageLoop pointers and always using C...

chrome/browser/net/dns_master.h

 // Copyright (c) 2006-2008 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.
 
 // A DnsMaster object is instantiated once in the browser
 // process, and manages asynchronous resolution of DNS hostnames.
 // Most hostname lists are sent out by renderer processes, and
 // involve lists of hostnames that *might* be used in the near
 // future by the browsing user.  The goal of this class is to
 // cause the underlying DNS structure to lookup a hostname before
@@ skipping 14 matching lines @@
 #include "chrome/browser/net/referrer.h"
 #include "chrome/common/net/dns.h"
 #include "testing/gtest/include/gtest/gtest_prod.h"
 
 using base::TimeDelta;
 
 namespace net {
 class HostResolver;
 }
 
-class MessageLoop;
-
 namespace chrome_browser_net {
 
 typedef chrome_common_net::NameList NameList;
 typedef std::map<std::string, DnsHostInfo> Results;
 
 class DnsMaster : public base::RefCountedThreadSafe<DnsMaster> {
  public:
   // |max_concurrent| specifies how many concurrent (paralell) prefetches will
-  // be performed. Host lookups will be issued on the |host_resolver_loop|
-  // thread, using the |host_resolver| instance.
-  DnsMaster(net::HostResolver* host_resolver, MessageLoop* host_resolver_loop,
+  // be performed. Host lookups will be issued on the IO thread, using the
+  // |host_resolver| instance.
+  DnsMaster(net::HostResolver* host_resolver,
             TimeDelta max_queue_delay_ms, size_t max_concurrent);
   ~DnsMaster();
 
   // Cancel pending requests and prevent new ones from being made.
   void Shutdown();
 
   // In some circumstances, for privacy reasons, all results should be
   // discarded.  This method gracefully handles that activity.
   // Destroy all our internal state, which shows what names we've looked up, and
   // how long each has taken, etc. etc.  We also destroy records of suggesses
@@ skipping 171 matching lines @@
   Results cache_eviction_map_;
 
   // The number of concurrent lookups currently allowed.
   const size_t max_concurrent_lookups_;
 
   // The maximum queueing delay that is acceptable before we enter congestion
   // reduction mode, and discard all queued (but not yet assigned) resolutions.
   const TimeDelta max_queue_delay_;
 
   // The host resovler we warm DNS entries for. The resolver (which is not
-  // thread safe) should be accessed only on |host_resolver_loop_|.
+  // thread safe) should be accessed only on the IO thread.
   scoped_refptr<net::HostResolver> host_resolver_;
-  MessageLoop* host_resolver_loop_;
 
   DISALLOW_COPY_AND_ASSIGN(DnsMaster);
 };
 
 }  // namespace chrome_browser_net
 
 #endif  // CHROME_BROWSER_NET_DNS_MASTER_H_