Add UMA to track parallel download fail reason |
||
Issue descriptionIt would be good to do the following analysis for parallel download: x% of the download is non-parallel because of no Accept-Ranges header in the response header or other criteria. There are about 5 criteria that we use to determine if a download can be parallel or not.
,
Apr 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c97ad191cb505a2e58a726a0aa5457668ae2477d commit c97ad191cb505a2e58a726a0aa5457668ae2477d Author: xingliu <xingliu@chromium.org> Date: Mon Apr 10 23:52:44 2017 Add UMA to track why a download is non-parallel when enabled parallel downloading. Move the function to determine parallel download back to DownloadJobFactory, since now it contains UMA code and should be only called in one place. BUG= 709295 Review-Url: https://codereview.chromium.org/2806653002 Cr-Commit-Position: refs/heads/master@{#463452} [modify] https://crrev.com/c97ad191cb505a2e58a726a0aa5457668ae2477d/content/browser/download/download_job_factory.cc [modify] https://crrev.com/c97ad191cb505a2e58a726a0aa5457668ae2477d/content/browser/download/download_stats.cc [modify] https://crrev.com/c97ad191cb505a2e58a726a0aa5457668ae2477d/content/browser/download/download_stats.h [modify] https://crrev.com/c97ad191cb505a2e58a726a0aa5457668ae2477d/content/browser/download/parallel_download_utils.cc [modify] https://crrev.com/c97ad191cb505a2e58a726a0aa5457668ae2477d/content/browser/download/parallel_download_utils.h [modify] https://crrev.com/c97ad191cb505a2e58a726a0aa5457668ae2477d/tools/metrics/histograms/histograms.xml
,
Apr 26 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by xingliu@chromium.org
, Apr 7 2017