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

Unified Diff: content/browser/renderer_host/pepper_file_message_filter.cc

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/renderer_host/pepper_file_message_filter.cc
diff --git a/content/browser/renderer_host/pepper_file_message_filter.cc b/content/browser/renderer_host/pepper_file_message_filter.cc
index 722db59fa749d4daf5608d183480e8cc89a5fb25..10b3430ee2ddba901721809bbcd8fd8ccbf1b455 100644
--- a/content/browser/renderer_host/pepper_file_message_filter.cc
+++ b/content/browser/renderer_host/pepper_file_message_filter.cc
@@ -44,11 +44,6 @@ PepperFileMessageFilter::PepperFileMessageFilter(
pepper_path_ = GetDataDirName(browser_context->GetPath());
}
-PepperFileMessageFilter::~PepperFileMessageFilter() {
- // This function should be called on the IO thread.
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
-}
-
void PepperFileMessageFilter::OverrideThreadForMessage(
const IPC::Message& message,
BrowserThread::ID* thread) {
@@ -80,6 +75,11 @@ FilePath PepperFileMessageFilter::GetDataDirName(const FilePath& profile_path) {
return profile_path.Append(FILE_PATH_LITERAL("Pepper Data"));
}
+PepperFileMessageFilter::~PepperFileMessageFilter() {
+ // This function should be called on the IO thread.
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+}
+
// Called on the FILE thread:
void PepperFileMessageFilter::OnOpenFile(
const webkit::ppapi::PepperFilePath& path,
« no previous file with comments | « content/browser/renderer_host/pepper_file_message_filter.h ('k') | content/browser/renderer_host/pepper_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698