RefCounted types should not have public destructors, delegate cleanup

chrome/service/cloud_print/cloud_print_url_fetcher.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 CHROME_SERVICE_CLOUD_PRINT_CLOUD_PRINT_URL_FETCHER_H_
 #define CHROME_SERVICE_CLOUD_PRINT_CLOUD_PRINT_URL_FETCHER_H_
 #pragma once
 
 #include <string>
 
@@ skipping 23 matching lines @@
       public net::URLFetcherDelegate {
  public:
   enum ResponseAction {
     CONTINUE_PROCESSING,
     STOP_PROCESSING,
     RETRY_REQUEST,
   };
 
   class Delegate {
    public:
-    virtual ~Delegate() { }
     // Override this to handle the raw response as it is available. No response
     // error checking is done before this method is called. If the delegate
     // returns CONTINUE_PROCESSING, we will then check for network
     // errors. Most implementations will not override this.
     virtual ResponseAction HandleRawResponse(
         const net::URLFetcher* source,
         const GURL& url,
         const net::URLRequestStatus& status,
         int response_code,
         const net::ResponseCookies& cookies,
@@ skipping 28 matching lines @@
     // HandleRawResponse returns CONTINUE_PROCESSING).
     // Returning RETRY_REQUEST will retry current request. (auth information
     // may have been updated and new info is available through the
     // Authenticator interface).
     // Returning CONTINUE_PROCESSING will treat auth error as a network error.
     virtual ResponseAction OnRequestAuthError() = 0;
 
     // Authentication information may change between retries.
     // CloudPrintURLFetcher will request auth info before sending any request.
     virtual std::string GetAuthHeader() = 0;
+
+   protected:
+    virtual ~Delegate() {}
   };
   CloudPrintURLFetcher();
 
   bool IsSameRequest(const net::URLFetcher* source);
 
   void StartGetRequest(const GURL& url,
                        Delegate* delegate,
                        int max_retries,
                        const std::string& additional_headers);
   void StartPostRequest(const GURL& url,
@@ skipping 28 matching lines @@
   int num_retries_;
   net::URLFetcher::RequestType request_type_;
   std::string additional_headers_;
   std::string post_data_mime_type_;
   std::string post_data_;
 };
 
 typedef CloudPrintURLFetcher::Delegate CloudPrintURLFetcherDelegate;
 
 #endif  // CHROME_SERVICE_CLOUD_PRINT_CLOUD_PRINT_URL_FETCHER_H_