RefCounted types should not have public destructors, net/

net/proxy/proxy_resolver_js_bindings.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_resolver_js_bindings.h"
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/string_util.h"
 #include "base/values.h"
@@ skipping 25 matching lines @@
         message_(message) {
   }
 
   virtual Value* ToValue() const OVERRIDE {
     DictionaryValue* dict = new DictionaryValue();
     dict->SetInteger("line_number", line_number_);
     dict->SetString("message", message_);
     return dict;
   }
 
+ protected:
+  virtual ~ErrorNetlogParams() {}
+
  private:
   const int line_number_;
   const string16 message_;
 
   DISALLOW_COPY_AND_ASSIGN(ErrorNetlogParams);
 };
 
 // Event parameters for a PAC alert().
 class AlertNetlogParams : public NetLog::EventParameters {
  public:
-  explicit AlertNetlogParams(const string16& message) : message_(message) {
-  }
+  explicit AlertNetlogParams(const string16& message) : message_(message) {}
 
   virtual Value* ToValue() const OVERRIDE {
     DictionaryValue* dict = new DictionaryValue();
     dict->SetString("message", message_);
     return dict;
   }
 
+ protected:
+  virtual ~AlertNetlogParams() {}
+
  private:
   const string16 message_;
 
   DISALLOW_COPY_AND_ASSIGN(AlertNetlogParams);
 };
 
 // ProxyResolverJSBindings implementation.
 class DefaultJSBindings : public ProxyResolverJSBindings {
  public:
   DefaultJSBindings(SyncHostResolver* host_resolver,
@@ skipping 238 matching lines @@
 
 // static
 ProxyResolverJSBindings* ProxyResolverJSBindings::CreateDefault(
     SyncHostResolver* host_resolver,
     NetLog* net_log,
     ProxyResolverErrorObserver* error_observer) {
   return new DefaultJSBindings(host_resolver, net_log, error_observer);
 }
 
 }  // namespace net