Improve download performance statistics.

content/browser/download/download_resource_handler.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_resource_handler.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 331 matching lines @@
           reason = content::DOWNLOAD_INTERRUPT_REASON_SERVER_PRECONDITION;
           break;
         default:
           reason = content::DOWNLOAD_INTERRUPT_REASON_SERVER_FAILED;
           break;
       }
     }
   }
 
   download_stats::RecordAcceptsRanges(accept_ranges_, bytes_read_);
+  download_stats::RecordNetworkBlockage(
+      base::TimeTicks::Now() - download_start_time_, total_pause_time_);
 
   CallStartedCB(DownloadId(), error_code);
 
   // Send the info down the stream.  Conditional is in case we get
   // OnResponseCompleted without OnResponseStarted.
   if (stream_writer_.get())
     stream_writer_->Close(reason);
 
   stream_writer_.reset();  // We no longer need the stream.
   read_buffer_ = NULL;
 
-  // Stats
-  download_stats::RecordNetworkBandwidth(
-      bytes_read_, base::TimeTicks::Now() - download_start_time_,
-      total_pause_time_);
-
   return true;
 }
 
 // If the content-length header is not present (or contains something other
 // than numbers), the incoming content_length is -1 (unknown size).
 // Set the content length to 0 to indicate unknown size to DownloadManager.
 void DownloadResourceHandler::SetContentLength(const int64& content_length) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   content_length_ = 0;
   if (content_length > 0)
@@ skipping 68 matching lines @@
   // false somewhere in the chain of resource handlers.
   CallStartedCB(DownloadId(), net::ERR_ACCESS_DENIED);
 
   // Remove output stream callback if a stream exists.
   if (stream_writer_.get())
     stream_writer_->RegisterCallback(base::Closure());
 
   UMA_HISTOGRAM_TIMES("SB2.DownloadDuration",
                       base::TimeTicks::Now() - download_start_time_);
 }