net: Add NET_API to net/base

net/base/host_resolver_impl.h

 // Copyright (c) 2011 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_BASE_HOST_RESOLVER_IMPL_H_
 #define NET_BASE_HOST_RESOLVER_IMPL_H_
 #pragma once
 
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/non_thread_safe.h"
 #include "base/time.h"
 #include "net/base/capturing_net_log.h"
 #include "net/base/host_cache.h"
 #include "net/base/host_resolver.h"
 #include "net/base/host_resolver_proc.h"
+#include "net/base/net_api.h"
 #include "net/base/net_log.h"
 #include "net/base/network_change_notifier.h"
 
 namespace net {
 
 // For each hostname that is requested, HostResolver creates a
 // HostResolverImpl::Job. This job gets dispatched to a thread in the global
 // WorkerPool, where it runs SystemHostResolverProc(). If requests for that same
 // host are made while the job is already outstanding, then they are attached
 // to the existing job rather than creating a new one. This avoids doing
@@ skipping 21 matching lines @@
 // threads.
 //
 // Requests are ordered in the queue based on their priority.
 //
 // Whenever we try to resolve the host, we post a delayed task to check if host
 // resolution (OnLookupComplete) is completed or not. If the original attempt
 // hasn't completed, then we start another attempt for host resolution. We take
 // the results from the first attempt that finishes and ignore the results from
 // all other attempts.
 
-class HostResolverImpl : public HostResolver,
-                         public base::NonThreadSafe,
-                         public NetworkChangeNotifier::IPAddressObserver {
+MSVC_PUSH_DISABLE_WARNING(4275)
+class NET_API HostResolverImpl

[wtc, 2011/05/16 20:55:17]

Can this be NET_TEST?  "Impl" seems to imply it is an internal
class.

[rvargas (doing something else), 2011/05/16 22:41:56]

Yup. It is used by four different files in chrome/content

+    : public HostResolver,
+      public base::NonThreadSafe,
+      public NetworkChangeNotifier::IPAddressObserver {
+MSVC_POP_WARNING()
  public:
   // The index into |job_pools_| for the various job pools. Pools with a higher
   // index have lower priority.
   //
   // Note: This is currently unused, since there is a single pool
   //       for all requests.
   enum JobPoolIndex {
     POOL_NORMAL = 0,
     POOL_COUNT,
   };
@@ skipping 264 matching lines @@
   HostResolverFlags additional_resolver_flags_;
 
   NetLog* net_log_;
 
   DISALLOW_COPY_AND_ASSIGN(HostResolverImpl);
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_HOST_RESOLVER_IMPL_H_