Add http response code metric in download system |
|||||
Issue descriptionCurrently we notice an issue according to metric data that parallel download feature has a higher SERVER_FAIL interruption type. we would like to know more details about the http response code from the server to analyze the cause. This metric only applies to download requests and will also be helpful to track general download system health.
,
May 6 2017
,
May 7 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/49b3271814acb74884e3781628c155775b7f7a0a commit 49b3271814acb74884e3781628c155775b7f7a0a Author: Xing Liu <xingliu@chromium.org> Date: Mon May 08 16:51:22 2017 Add a metric to record http response code for download requests. Currently we notice an issue according to metric data that parallel download feature has a higher SERVER_FAIL interruption type. we would like to know more details about the http response code from the server to analyze the cause. This metric only applies to download requests and will also be helpful to track general download system health. BUG= 718465 Review-Url: https://codereview.chromium.org/2862743002 Cr-Commit-Position: refs/heads/master@{#469888} (cherry picked from commit 503e20c58c8fdae480d5e7bbc538365f17e32fa1) Review-Url: https://codereview.chromium.org/2867803002 . Cr-Commit-Position: refs/branch-heads/3071@{#449} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} [modify] https://crrev.com/49b3271814acb74884e3781628c155775b7f7a0a/content/browser/download/download_request_core.cc [modify] https://crrev.com/49b3271814acb74884e3781628c155775b7f7a0a/content/browser/download/download_stats.cc [modify] https://crrev.com/49b3271814acb74884e3781628c155775b7f7a0a/content/browser/download/download_stats.h [modify] https://crrev.com/49b3271814acb74884e3781628c155775b7f7a0a/tools/metrics/histograms/histograms.xml
,
May 11 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bugdroid1@chromium.org
, May 6 2017