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

Unified Diff: content/browser/download/drag_download_util.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/download/drag_download_util.cc
diff --git a/content/browser/download/drag_download_util.cc b/content/browser/download/drag_download_util.cc
index b68ca16bb961c453d17c4025c0e76dbe48bc95b1..a62bbe438f055f30ef5f144c292c6ee086160936 100644
--- a/content/browser/download/drag_download_util.cc
+++ b/content/browser/download/drag_download_util.cc
@@ -98,13 +98,6 @@ PromiseFileFinalizer::PromiseFileFinalizer(
: drag_file_downloader_(drag_file_downloader) {
}
-PromiseFileFinalizer::~PromiseFileFinalizer() {}
-
-void PromiseFileFinalizer::Cleanup() {
- if (drag_file_downloader_.get())
- drag_file_downloader_ = NULL;
-}
-
void PromiseFileFinalizer::OnDownloadCompleted(const FilePath& file_path) {
BrowserThread::PostTask(
BrowserThread::UI, FROM_HERE,
@@ -117,4 +110,11 @@ void PromiseFileFinalizer::OnDownloadAborted() {
base::Bind(&PromiseFileFinalizer::Cleanup, this));
}
+PromiseFileFinalizer::~PromiseFileFinalizer() {}
+
+void PromiseFileFinalizer::Cleanup() {
+ if (drag_file_downloader_.get())
+ drag_file_downloader_ = NULL;
+}
+
} // namespace drag_download_util
« no previous file with comments | « content/browser/download/drag_download_util.h ('k') | content/browser/fileapi/chrome_blob_storage_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698