RefCounted types should not have public destructors, net/ edition

net/socket_stream/socket_stream.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 NET_SOCKET_STREAM_SOCKET_STREAM_H_
 #define NET_SOCKET_STREAM_SOCKET_STREAM_H_
 #pragma once
 
 #include <deque>
 #include <map>
@@ skipping 41 matching lines @@
   // information with a SocketStream.  Use GetUserData(key) and
   // SetUserData(key, data).
   class UserData {
    public:
     UserData() {}
     virtual ~UserData() {}
   };
 
   class NET_EXPORT Delegate {
    public:
-    virtual ~Delegate() {}
-
     virtual int OnStartOpenConnection(SocketStream* socket,
                                       const CompletionCallback& callback) {
       return OK;
     }
 
     // Called when socket stream has been connected.  The socket stream accepts
     // at most |max_pending_send_allowed| so that a client of the socket stream
     // should keep track of how much it has pending and shouldn't go over
     // |max_pending_send_allowed| bytes.
     virtual void OnConnected(SocketStream* socket,
@@ skipping 40 matching lines @@
     }
 
     // Called when a cookie is set to allow the delegate to block access to the
     // cookie.
     virtual bool CanSetCookie(SocketStream* request,
                               const GURL& url,
                               const std::string& cookie_line,
                               CookieOptions* options) {
       return true;
     }
+
+   protected:
+    virtual ~Delegate() {}
   };
 
   SocketStream(const GURL& url, Delegate* delegate);
 
   // The user data allows the clients to associate data with this job.
   // Multiple user data values can be stored under different keys.
   // This job will TAKE OWNERSHIP of the given data pointer, and will
   // delete the object if it is changed or the job is destroyed.
   UserData* GetUserData(const void* key) const;
   void SetUserData(const void* key, UserData* data);
@@ skipping 240 matching lines @@
   bool server_closed_;
 
   scoped_ptr<SocketStreamMetrics> metrics_;
 
   DISALLOW_COPY_AND_ASSIGN(SocketStream);
 };
 
 }  // namespace net
 
 #endif  // NET_SOCKET_STREAM_SOCKET_STREAM_H_