RefCounted types should not have public destructors, delegate cleanup

chrome/browser/cancelable_request.h

Index: chrome/browser/cancelable_request.h
diff --git a/chrome/browser/cancelable_request.h b/chrome/browser/cancelable_request.h
index 3ab893c4d4e4782298e703bf9c0c1cd2a9fd852f..69939ede6e39704a17985c7bde9139aa8d3f29d9 100644
--- a/chrome/browser/cancelable_request.h
+++ b/chrome/browser/cancelable_request.h
@@ -122,7 +122,6 @@ class CancelableRequestProvider {
   typedef int Handle;
 
   CancelableRequestProvider();
-  virtual ~CancelableRequestProvider();
 
   // Called by the enduser of the request to cancel it. This MUST be called on
   // the same thread that originally issued the request (which is also the same
@@ -131,6 +130,8 @@ class CancelableRequestProvider {
   void CancelRequest(Handle handle);
 
  protected:
+  virtual ~CancelableRequestProvider();
+
   // Adds a new request and initializes it. This is called by a derived class
   // to add a new request. The request's Init() will be called (which is why
   // the consumer is required. The handle to the new request is returned.