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

Unified Diff: content/test/net/url_request_slow_download_job.cc

Issue 10074001: Initial implementation of the ByteStream refactor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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/test/net/url_request_slow_download_job.cc
diff --git a/content/test/net/url_request_slow_download_job.cc b/content/test/net/url_request_slow_download_job.cc
index 3ba1be72a950bc33a9a758543ac0a1448593d1a5..ba32d2364dc7d333b6e2e5cf1b10204f1e04f4cc 100644
--- a/content/test/net/url_request_slow_download_job.cc
+++ b/content/test/net/url_request_slow_download_job.cc
@@ -185,6 +185,8 @@ void URLRequestSlowDownloadJob::CheckDoneStatus() {
ReadStatus status = FillBufferHelper(buffer_, buffer_size_, &bytes_written);
DCHECK_EQ(BUFFER_FILLED, status);
SetStatus(net::URLRequestStatus());
+ // Release reference
ahendrickson 2012/04/16 15:14:27 Is this related to this CL?
Randy Smith (Not in Mondays) 2012/04/18 19:10:38 Sorta-not-really. It's an already existing proble
+ buffer_ = NULL;
NotifyReadComplete(bytes_written);
} else {
MessageLoop::current()->PostDelayedTask(
« content/browser/download/download_resource_handler.cc ('K') | « content/content_tests.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698