Use ByteStream in downloads system to decouple source and sink.

content/browser/download/base_file.cc

Index: content/browser/download/base_file.cc
diff --git a/content/browser/download/base_file.cc b/content/browser/download/base_file.cc
index 9516fb3c5d16e6fed89cc5744787818e0d103a7c..05dc9d3f59296fc3e5abfac5ef4c74464d51a5d5 100644
--- a/content/browser/download/base_file.cc
+++ b/content/browser/download/base_file.cc
@@ -272,13 +272,6 @@ net::Error BaseFile::AppendDataToFile(const char* data, size_t data_len) {
         download_stats::APPEND_TO_DETACHED_FILE_COUNT);
   }
 
-  if (bound_net_log_.IsLoggingAllEvents()) {
-    bound_net_log_.AddEvent(
-        net::NetLog::TYPE_DOWNLOAD_FILE_WRITTEN,
-        make_scoped_refptr(new net::NetLogIntegerParameter(
-            "byte_count", data_len)));

[Randy Smith (Not in Mondays), 2012/05/19 15:29:57]

For downloads, this was redundant with logging occurring now in
download_file_impl.cc, and I don't think we've found it useful to date for
SavePackage problems.  It didn't seem worth the extra effort to put in parallel
logging for SavePackage for a bit of logging that was only recently inserted for
downloads.

[Randy Smith (Not in Mondays), 2012/05/23 03:33:15]

Done.

-  }
-
   if (!file_stream_.get())
     return LOG_ERROR("get", net::ERR_INVALID_HANDLE);