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

Unified Diff: webkit/blob/blob_url_request_job.cc

Issue 10066044: RefCounted types should not have public destructors, webkit/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Implementation ordering 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
« no previous file with comments | « webkit/blob/blob_url_request_job.h ('k') | webkit/database/database_quota_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/blob/blob_url_request_job.cc
diff --git a/webkit/blob/blob_url_request_job.cc b/webkit/blob/blob_url_request_job.cc
index 65f8e0855d944c57366be5491f962560e90d080b..6f8ef92b250a7bfd97ddf72845224340e176bc22 100644
--- a/webkit/blob/blob_url_request_job.cc
+++ b/webkit/blob/blob_url_request_job.cc
@@ -64,10 +64,6 @@ BlobURLRequestJob::BlobURLRequestJob(
DCHECK(file_thread_proxy_);
}
-BlobURLRequestJob::~BlobURLRequestJob() {
- STLDeleteValues(&index_to_reader_);
-}
-
void BlobURLRequestJob::Start() {
// Continue asynchronously.
MessageLoop::current()->PostTask(
@@ -151,6 +147,10 @@ void BlobURLRequestJob::SetExtraRequestHeaders(
}
}
+BlobURLRequestJob::~BlobURLRequestJob() {
+ STLDeleteValues(&index_to_reader_);
+}
+
void BlobURLRequestJob::DidStart() {
// We only support GET request per the spec.
if (request()->method() != "GET") {
« no previous file with comments | « webkit/blob/blob_url_request_job.h ('k') | webkit/database/database_quota_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698