RefCounted types should not have public destructors, content/browser part 1

content/browser/renderer_host/socket_stream_dispatcher_host.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 CONTENT_BROWSER_RENDERER_HOST_SOCKET_STREAM_DISPATCHER_HOST_H_
 #define CONTENT_BROWSER_RENDERER_HOST_SOCKET_STREAM_DISPATCHER_HOST_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 18 matching lines @@
 // It also acts as SocketStream::Delegate so that it sends
 // ViewMsg_SocketStream_* messages back to renderer.
 class SocketStreamDispatcherHost : public content::BrowserMessageFilter,
                                    public net::SocketStream::Delegate,
                                    public SSLErrorHandler::Delegate {
  public:
   SocketStreamDispatcherHost(
       int render_process_id,
       ResourceMessageFilter::URLRequestContextSelector* selector,
       content::ResourceContext* resource_context);
-  virtual ~SocketStreamDispatcherHost();
 
   // content::BrowserMessageFilter:
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok) OVERRIDE;
 
   // The object died, so cancel and detach all requests associated with it.
   void CancelRequestsForProcess(int host_id);
 
   // SocketStream::Delegate:
   virtual void OnConnected(net::SocketStream* socket,
@@ skipping 11 matching lines @@
                             const GURL& url,
                             const std::string& cookie_line,
                             net::CookieOptions* options) OVERRIDE;
 
   // SSLErrorHandler::Delegate methods:
   virtual void CancelSSLRequest(const content::GlobalRequestID& id,
                                 int error,
                                 const net::SSLInfo* ssl_info) OVERRIDE;
   virtual void ContinueSSLRequest(const content::GlobalRequestID& id) OVERRIDE;
 
+ protected:
+  virtual ~SocketStreamDispatcherHost();
+
  private:
   // Message handlers called by OnMessageReceived.
   void OnConnect(int render_view_id, const GURL& url, int socket_id);
   void OnSendData(int socket_id, const std::vector<char>& data);
   void OnCloseReq(int socket_id);
 
   void DeleteSocketStreamHost(int socket_id);
 
   net::URLRequestContext* GetURLRequestContext();
 
   IDMap<SocketStreamHost> hosts_;
   int render_process_id_;
   const scoped_ptr<ResourceMessageFilter::URLRequestContextSelector>
       url_request_context_selector_;
   content::ResourceContext* resource_context_;
 
   DISALLOW_COPY_AND_ASSIGN(SocketStreamDispatcherHost);
 };
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_SOCKET_STREAM_DISPATCHER_HOST_H_