RefCounted types should not have public destructors, net/

net/proxy/proxy_service.cc

 // 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.
 
 #include "net/proxy/proxy_service.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 331 matching lines @@
     DictionaryValue* dict = new DictionaryValue();
     // The "old_config" is optional -- the first notification will not have
     // any "previous" configuration.
     if (old_config_.is_valid())
       dict->Set("old_config", old_config_.ToValue());
     dict->Set("new_config", new_config_.ToValue());
     return dict;
   }
 
  private:
+  virtual ~ProxyConfigChangedNetLogParam() {}
+
   const ProxyConfig old_config_;
   const ProxyConfig new_config_;
   DISALLOW_COPY_AND_ASSIGN(ProxyConfigChangedNetLogParam);
 };
 
 class BadProxyListNetLogParam : public NetLog::EventParameters {
  public:
   BadProxyListNetLogParam(const ProxyRetryInfoMap& retry_info) {
     proxy_list_.reserve(retry_info.size());
     for (ProxyRetryInfoMap::const_iterator iter = retry_info.begin();
          iter != retry_info.end(); ++iter) {
       proxy_list_.push_back(iter->first);
     }
   }
 
   virtual Value* ToValue() const OVERRIDE {
     DictionaryValue* dict = new DictionaryValue();
     ListValue* list = new ListValue();
     for (std::vector<std::string>::const_iterator iter = proxy_list_.begin();
          iter != proxy_list_.end(); ++iter)
       list->Append(Value::CreateStringValue(*iter));
     dict->Set("bad_proxy_list", list);
     return dict;
   }
 
  private:
+  virtual ~BadProxyListNetLogParam() {}
+
   std::vector<std::string> proxy_list_;
   DISALLOW_COPY_AND_ASSIGN(BadProxyListNetLogParam);
 };
 
 }  // namespace
 
 // ProxyService::InitProxyResolver --------------------------------------------
 
 // This glues together two asynchronous steps:
 //   (1) ProxyScriptDecider -- try to fetch/validate a sequence of PAC scripts
@@ skipping 1229 matching lines @@
     OnCompletion(result_);
   }
 }
 
 void SyncProxyServiceHelper::OnCompletion(int rv) {
   result_ = rv;
   event_.Signal();
 }
 
 }  // namespace net