RefCounted types should not have public destructors, chrome/browser/ part 1

chrome/browser/history/shortcuts_database.h

Index: chrome/browser/history/shortcuts_database.h
diff --git a/chrome/browser/history/shortcuts_database.h b/chrome/browser/history/shortcuts_database.h
index 2d73d22696cb2a20dcbd6f36bdded5d83f1889ba..11d4caac9e8817af7b6d65afd134d1e59faf840d 100644
--- a/chrome/browser/history/shortcuts_database.h
+++ b/chrome/browser/history/shortcuts_database.h
@@ -43,7 +43,6 @@ class ShortcutsDatabase : public base::RefCountedThreadSafe<ShortcutsDatabase> {
   typedef std::map<std::string, ShortcutsBackend::Shortcut> GuidToShortcutMap;
 
   explicit ShortcutsDatabase(const FilePath& folder_path);
-  virtual ~ShortcutsDatabase();
 
   bool Init();
 
@@ -66,6 +65,10 @@ class ShortcutsDatabase : public base::RefCountedThreadSafe<ShortcutsDatabase> {
   bool LoadShortcuts(GuidToShortcutMap* shortcuts);
 
  private:
+  friend class base::RefCountedThreadSafe<ShortcutsDatabase>;
+
+  virtual ~ShortcutsDatabase();
+
   // Ensures that the table is present.
   bool EnsureTable();