RefCounted types should not have public destructors, delegate cleanup

chrome/browser/history/history.h

Index: chrome/browser/history/history.h
diff --git a/chrome/browser/history/history.h b/chrome/browser/history/history.h
index adfa4be61d95d6eb718607a7fc767ddab27bf70f..45e312536a04a2698cbcaa679b944cffc4b1b890 100644
--- a/chrome/browser/history/history.h
+++ b/chrome/browser/history/history.h
@@ -253,8 +253,6 @@ class HistoryService : public CancelableRequestProvider,
   // should handle this appropriately.
   class URLEnumerator {
    public:
-    virtual ~URLEnumerator() {}
-
     // Indicates that a URL is available. There will be exactly one call for
     // every URL in history.
     virtual void OnURL(const GURL& url) = 0;
@@ -263,6 +261,9 @@ class HistoryService : public CancelableRequestProvider,
     // more callbacks made. This call is guaranteed to occur, even if there are
     // no URLs. If all URLs were iterated, success will be true.
     virtual void OnComplete(bool success) = 0;
+
+   protected:
+    virtual ~URLEnumerator() {}
   };
 
   // Enumerate all URLs in history. The given iterator will be owned by the