Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(991)

Unified Diff: chrome/browser/history/shortcuts_database.h

Issue 10071032: RefCounted types should not have public destructors, chrome/browser/ part 1 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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();
sky 2012/04/13 15:50:52 Move implementation to match.
+
// Ensures that the table is present.
bool EnsureTable();

Powered by Google App Engine
This is Rietveld 408576698