RefCounted types should not have public destructors, net/ edition

net/base/network_change_notifier.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_BASE_NETWORK_CHANGE_NOTIFIER_H_
 #define NET_BASE_NETWORK_CHANGE_NOTIFIER_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/observer_list_threadsafe.h"
@@ skipping 34 matching lines @@
   CONNECTION_ETHERNET,
   CONNECTION_WIFI,
   CONNECTION_2G,
   CONNECTION_3G,
   CONNECTION_4G,
   CONNECTION_NONE     // No connection.
 };
 
   class NET_EXPORT IPAddressObserver {
    public:
-    virtual ~IPAddressObserver() {}
-
     // Will be called when the IP address of the primary interface changes.
     // This includes when the primary interface itself changes.
     virtual void OnIPAddressChanged() = 0;
 
    protected:
     IPAddressObserver() {}
+    virtual ~IPAddressObserver() {}
 
    private:
     DISALLOW_COPY_AND_ASSIGN(IPAddressObserver);
   };
 
   class NET_EXPORT ConnectionTypeObserver {
    public:
-    virtual ~ConnectionTypeObserver() {}
-
-  // Will be called when the connection type of the system has changed.
-  // See NetworkChangeNotifier::GetConnectionType() for important caveats
-  // about the unreliability of using this signal to infer the ability to
-  // reach remote sites.
+    // Will be called when the connection type of the system has changed.
+    // See NetworkChangeNotifier::GetConnectionType() for important caveats
+    // about the unreliability of using this signal to infer the ability to
+    // reach remote sites.
     virtual void OnConnectionTypeChanged(ConnectionType type) = 0;
 
    protected:
     ConnectionTypeObserver() {}
+    virtual ~ConnectionTypeObserver() {}
 
    private:
     DISALLOW_COPY_AND_ASSIGN(ConnectionTypeObserver);
   };
 
   class NET_EXPORT DNSObserver {
    public:
-    virtual ~DNSObserver() {}
-
     // Will be called when the DNS settings of the system may have changed.
     // The flags set in |detail| provide the specific set of changes.
     virtual void OnDNSChanged(unsigned detail) = 0;
 
    protected:
     DNSObserver() {}
+    virtual ~DNSObserver() {}
 
    private:
     DISALLOW_COPY_AND_ASSIGN(DNSObserver);
   };
 
   virtual ~NetworkChangeNotifier();
 
   // See the description of NetworkChangeNotifier::GetConnectionType().
   // Implementations must be thread-safe. Implementations must also be
   // cheap as this could be called (repeatedly) from the IO thread.
@@ skipping 109 matching lines @@
   //             http://crbug.com/116139
   base::Lock watching_dns_lock_;
   bool watching_dns_;
 
   DISALLOW_COPY_AND_ASSIGN(NetworkChangeNotifier);
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_CHANGE_NOTIFIER_H_