Add new UMA stats for parallelizable download

Desktop / Chromium - qinmin [chromium.org] - 20 April 2017 17:04 EDT

To study the effectiveness of parallel download, we want to compare the bandwidth if parallel downloads are not parallelized. Because parallel download has requirements on file size, so comparing with all other downloads is be fair. This CL records the UMA metrics for downloads that satisfy the requirement of parallel download, while don't use parallel requests.

BUG=644352,712892

Review-Url: https://codereview.chromium.org/2823273004 Cr-Commit-Position: refs/heads/master@{#466128}

2269e30 Add new UMA stats for parallelizable download
content/browser/download/download_browsertest.cc | 5 +-
content/browser/download/download_file.h | 8 +--
content/browser/download/download_file_impl.cc | 17 +++--
content/browser/download/download_file_impl.h | 3 +-
content/browser/download/download_file_unittest.cc | 2 +-
content/browser/download/download_item_impl.cc | 33 ++++++----
.../download/download_item_impl_unittest.cc | 14 ++--
content/browser/download/download_job.cc | 6 +-
content/browser/download/download_job.h | 5 +-
content/browser/download/download_job_factory.cc | 26 ++++----
content/browser/download/download_job_impl.cc | 11 +++-
content/browser/download/download_job_impl.h | 7 +-
content/browser/download/download_stats.cc | 76 +++++++++++++++-------
content/browser/download/download_stats.h | 18 +++--
content/browser/download/mock_download_file.cc | 5 +-
content/browser/download/mock_download_file.h | 5 +-
content/browser/download/parallel_download_job.cc | 6 +-
content/browser/download/parallel_download_job.h | 1 -
.../browser/download/parallel_download_utils.cc | 6 +-
content/browser/download/parallel_download_utils.h | 4 ++
content/public/test/test_file_error_injector.cc | 8 ++-
tools/metrics/histograms/histograms.xml | 36 +++++++++-
22 files changed, 202 insertions(+), 100 deletions(-)

Upstream: git.chromium.org


  • Share