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

Unified Diff: chrome/browser/ui/webui/options2/certificate_manager_handler2.cc

Issue 10392152: RefCounted types should not have public destructors, Linux fixes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 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/ui/webui/options2/certificate_manager_handler2.cc
diff --git a/chrome/browser/ui/webui/options2/certificate_manager_handler2.cc b/chrome/browser/ui/webui/options2/certificate_manager_handler2.cc
index 5129a2178220d5ba45c968db2097e5ddfdaad76d..af91ea5ada9a421762fbdee04efefbfb1812adf8 100644
--- a/chrome/browser/ui/webui/options2/certificate_manager_handler2.cc
+++ b/chrome/browser/ui/webui/options2/certificate_manager_handler2.cc
@@ -174,6 +174,9 @@ class FileAccessProvider
const WriteCallback& callback);
private:
+ friend class base::RefCountedThreadSafe<FileAccessProvider>;
+ virtual ~FileAccessProvider() {}
+
void DoRead(scoped_refptr<CancelableRequest<ReadCallback> > request,
FilePath path);
void DoWrite(scoped_refptr<CancelableRequest<WriteCallback> > request,

Powered by Google App Engine
This is Rietveld 408576698