Improve download performance statistics.

content/browser/download/download_file_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/download/download_file_impl.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/file_util.h"
@@ skipping 165 matching lines @@
     switch (state) {
       case content::ByteStreamReader::STREAM_EMPTY:
         break;
       case content::ByteStreamReader::STREAM_HAS_DATA:
         {
           ++num_buffers;
           base::TimeTicks write_start(base::TimeTicks::Now());
           result = AppendDataToFile(
               incoming_data.get()->data(), incoming_data_size);
           disk_writes_time_ += (base::TimeTicks::Now() - write_start);
+          bytes_seen_ += incoming_data_size;
           total_incoming_data_size += incoming_data_size;
           reason = content::ConvertNetErrorToInterruptReason(
               result, content::DOWNLOAD_INTERRUPT_FROM_DISK);
         }
         break;
       case content::ByteStreamReader::STREAM_COMPLETE:
         {
           reason = stream_reader_->GetStatus();
           SendUpdate();
           base::TimeTicks close_start(base::TimeTicks::Now());
@@ skipping 18 matching lines @@
   if (state == content::ByteStreamReader::STREAM_HAS_DATA &&
       now - start > delta) {
     BrowserThread::PostTask(
         BrowserThread::FILE, FROM_HERE,
         base::Bind(&DownloadFileImpl::StreamActive,
                    weak_factory_.GetWeakPtr()));
   }
 
   if (total_incoming_data_size)
     download_stats::RecordFileThreadReceiveBuffers(num_buffers);
-  bytes_seen_ += total_incoming_data_size;
 
   download_stats::RecordContiguousWriteTime(now - start);
 
   // Take care of communication with our controller.
   if (reason != content::DOWNLOAD_INTERRUPT_REASON_NONE) {
     // Error case for both upstream source and file write.
     // Shut down processing and signal an error to our controller.
     // Our controller will clean us up.
     stream_reader_->RegisterCallback(base::Closure());
     weak_factory_.InvalidateWeakPtrs();
@@ skipping 29 matching lines @@
 
 void DownloadFileImpl::SendUpdate() {
   if (download_manager_.get()) {
     BrowserThread::PostTask(
         BrowserThread::UI, FROM_HERE,
         base::Bind(&DownloadManager::UpdateDownload,
                    download_manager_, id_.local(),
                    BytesSoFar(), CurrentSpeed(), GetHashState()));
   }
 }