RefCounted types should not have public destructors, delegate cleanup

content/browser/ssl/ssl_error_handler.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_SSL_SSL_ERROR_HANDLER_H_
 #define CONTENT_BROWSER_SSL_SSL_ERROR_HANDLER_H_
 #pragma once
 
 #include <string>
 
@@ skipping 42 matching lines @@
    public:
     // Called when SSLErrorHandler decides to cancel the request because of
     // the SSL error.
     virtual void CancelSSLRequest(const content::GlobalRequestID& id,
                                   int error,
                                   const net::SSLInfo* ssl_info) = 0;
 
     // Called when SSLErrorHandler decides to continue the request despite the
     // SSL error.
     virtual void ContinueSSLRequest(const content::GlobalRequestID& id) = 0;
+
+   protected:
+    virtual ~Delegate() {}
   };
 
   virtual SSLCertErrorHandler* AsSSLCertErrorHandler();
 
   // Find the appropriate SSLManager for the net::URLRequest and begin handling
   // this error.
   //
   // Call on UI thread.
   void Dispatch();
 
@@ skipping 87 matching lines @@
   const ResourceType::Type resource_type_;
 
   // A flag to make sure we notify the net::URLRequest exactly once.
   // Should only be accessed on the IO thread
   bool request_has_been_notified_;
 
   DISALLOW_COPY_AND_ASSIGN(SSLErrorHandler);
 };
 
 #endif  // CONTENT_BROWSER_SSL_SSL_ERROR_HANDLER_H_