RefCounted types should not have public destructors, net/

net/spdy/spdy_session.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/spdy/spdy_session.h"
 
 #include <map>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
@@ skipping 94 matching lines @@
 
 const int kReadBufferSize = 8 * 1024;
 const int kDefaultConnectionAtRiskOfLossSeconds = 10;
 const int kTrailingPingDelayTimeSeconds = 1;
 const int kHungIntervalSeconds = 10;
 
 class NetLogSpdySessionParameter : public NetLog::EventParameters {
  public:
   NetLogSpdySessionParameter(const HostPortProxyPair& host_pair)
       : host_pair_(host_pair) {}
+
   virtual Value* ToValue() const {
     DictionaryValue* dict = new DictionaryValue();
     dict->Set("host", new StringValue(host_pair_.first.ToString()));
     dict->Set("proxy", new StringValue(host_pair_.second.ToPacString()));
     return dict;
   }
+
  private:
+  virtual ~NetLogSpdySessionParameter() {}
+
   const HostPortProxyPair host_pair_;
   DISALLOW_COPY_AND_ASSIGN(NetLogSpdySessionParameter);
 };
 
 class NetLogSpdySettingParameter : public NetLog::EventParameters {
  public:
   explicit NetLogSpdySettingParameter(SpdySettingsIds id,
                                       SpdySettingsFlags flags,
                                       uint32 value)
       : id_(id),
         flags_(flags),
         value_(value) {
   }
 
   virtual Value* ToValue() const {
     DictionaryValue* dict = new DictionaryValue();
     dict->SetInteger("id", id_);
     dict->SetInteger("flags", flags_);
     dict->SetInteger("value", value_);
     return dict;
   }
 
  private:
-  ~NetLogSpdySettingParameter() {}
+  virtual ~NetLogSpdySettingParameter() {}
+
   const SpdySettingsIds id_;
   const SpdySettingsFlags flags_;
   const uint32 value_;
 
   DISALLOW_COPY_AND_ASSIGN(NetLogSpdySettingParameter);
 };
 
 class NetLogSpdySettingsParameter : public NetLog::EventParameters {
  public:
   explicit NetLogSpdySettingsParameter(const SpdySettings& settings)
@@ skipping 1804 matching lines @@
 SSLClientSocket* SpdySession::GetSSLClientSocket() const {
   if (!is_secure_)
     return NULL;
   SSLClientSocket* ssl_socket =
       reinterpret_cast<SSLClientSocket*>(connection_->socket());
   DCHECK(ssl_socket);
   return ssl_socket;
 }
 
 }  // namespace net