RefCounted types should not have public destructors, net/

net/proxy/proxy_config_service_mac.cc

 // 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.
 
 #include "net/proxy/proxy_config_service_mac.h"
 
 #include <CoreFoundation/CoreFoundation.h>
 #include <SystemConfiguration/SystemConfiguration.h>
 
 #include "base/bind.h"
@@ skipping 161 matching lines @@
   void Orphan() {
     parent_ = NULL;
   }
 
   void OnProxyConfigChanged(const ProxyConfig& new_config) {
     if (parent_)
       parent_->OnProxyConfigChanged(new_config);
   }
 
  private:
+  friend class base::RefCountedThreadSafe<Helper>;
+  ~Helper() {}
+
   ProxyConfigServiceMac* parent_;
 };
 
 ProxyConfigServiceMac::ProxyConfigServiceMac(MessageLoop* io_loop)
     : forwarder_(this),
       has_fetched_config_(false),
       helper_(new Helper(this)),
       io_loop_(io_loop) {
   DCHECK(io_loop);
   config_watcher_.reset(new NetworkConfigWatcherMac(&forwarder_));
@@ skipping 67 matching lines @@
   // Keep track of the last value we have seen.
   has_fetched_config_ = true;
   last_config_fetched_ = new_config;
 
   // Notify all the observers.
   FOR_EACH_OBSERVER(Observer, observers_,
                     OnProxyConfigChanged(new_config, CONFIG_VALID));
 }
 
 }  // namespace net