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

Unified Diff: content/browser/fileapi/fileapi_message_filter.h

Issue 10068037: RefCounted types should not have public destructors, content/browser part 1 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: MSVC fixes 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: content/browser/fileapi/fileapi_message_filter.h
diff --git a/content/browser/fileapi/fileapi_message_filter.h b/content/browser/fileapi/fileapi_message_filter.h
index 348520416523db60664787558c1835bd23ecc163..c6d94ed9fa2776bdece271940f0afd799569667a 100644
--- a/content/browser/fileapi/fileapi_message_filter.h
+++ b/content/browser/fileapi/fileapi_message_filter.h
@@ -53,7 +53,6 @@ class FileAPIMessageFilter : public content::BrowserMessageFilter {
net::URLRequestContext* request_context,
fileapi::FileSystemContext* file_system_context,
ChromeBlobStorageContext* blob_storage_context);
- virtual ~FileAPIMessageFilter();
// content::BrowserMessageFilter implementation.
virtual void OnChannelConnected(int32 peer_pid) OVERRIDE;
@@ -66,6 +65,9 @@ class FileAPIMessageFilter : public content::BrowserMessageFilter {
void UnregisterOperation(int request_id);
+ protected:
+ virtual ~FileAPIMessageFilter();
+
private:
void OnOpen(int request_id,
const GURL& origin_url,
« no previous file with comments | « content/browser/fileapi/chrome_blob_storage_context.cc ('k') | content/browser/fileapi/fileapi_message_filter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698