RefCounted types should not have public destructors, net/ edition

net/url_request/url_request.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_URL_REQUEST_URL_REQUEST_H_
 #define NET_URL_REQUEST_URL_REQUEST_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 215 matching lines @@
   //    - OnReadCompleted* (zero or more calls until all data is read)
   //
   // Read() must be called at least once. Read() returns true when it completed
   // immediately, and false if an IO is pending or if there is an error.  When
   // Read() returns false, the caller can check the Request's status() to see
   // if an error occurred, or if the IO is just pending.  When Read() returns
   // true with zero bytes read, it indicates the end of the response.
   //
   class NET_EXPORT Delegate {
    public:
-    virtual ~Delegate() {}
-
     // Called upon a server-initiated redirect.  The delegate may call the
     // request's Cancel method to prevent the redirect from being followed.
     // Since there may be multiple chained redirects, there may also be more
     // than one redirect call.
     //
     // When this function is called, the request will still contain the
     // original URL, the destination of the redirect is provided in 'new_url'.
     // If the delegate does not cancel the request and |*defer_redirect| is
     // false, then the redirect will be followed, and the request's URL will be
     // changed to the new URL.  Otherwise if the delegate does not cancel the
@@ skipping 47 matching lines @@
     virtual void OnResponseStarted(URLRequest* request) = 0;
 
     // Called when the a Read of the response body is completed after an
     // IO_PENDING status from a Read() call.
     // The data read is filled into the buffer which the caller passed
     // to Read() previously.
     //
     // If an error occurred, request->status() will contain the error,
     // and bytes read will be -1.
     virtual void OnReadCompleted(URLRequest* request, int bytes_read) = 0;
+
+   protected:
+    virtual ~Delegate() {}
   };
 
   // Initialize an URL request.
   URLRequest(const GURL& url, Delegate* delegate);
 
   // If destroyed after Start() has been called but while IO is pending,
   // then the request will be effectively canceled and the delegate
   // will not have any more of its methods called.
   virtual ~URLRequest();
 
@@ skipping 486 matching lines @@
   base::TimeTicks creation_time_;
 
   scoped_ptr<const base::debug::StackTrace> stack_trace_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequest);
 };
 
 }  // namespace net
 
 #endif  // NET_URL_REQUEST_URL_REQUEST_H_