New issue
Advanced search Search tips

Issue 712318 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Investigate high download interruption count

Project Member Reported by qin...@chromium.org, Apr 17 2017

Issue description

For some users, we are seeing 1000+ interruptions during download.

Need to investigate why this happens, whether it is a bug in UMA reporting, or real network condition issues.

This allows us to do better in throttling those download.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bbe9ad733bbdf501cc63db9b624cfed076d33c32

commit bbe9ad733bbdf501cc63db9b624cfed076d33c32
Author: qinmin <qinmin@chromium.org>
Date: Wed Apr 19 00:50:53 2017

Record more download resumption UMAs to analyze the how download is resumed

This CL adds the metrics to study how many resumption is attempted for a download.
This allows us to study the high download interruption counts.
And choose a better throttling strategy.

BUG= 712318 

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

[modify] https://crrev.com/bbe9ad733bbdf501cc63db9b624cfed076d33c32/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
[modify] https://crrev.com/bbe9ad733bbdf501cc63db9b624cfed076d33c32/tools/metrics/histograms/histograms.xml

Comment 2 by qin...@chromium.org, Apr 19 2017

Labels: Merge-Request-59 Type-Bug-Regression
Owner: qin...@chromium.org
Status: Started (was: Unconfirmed)
Project Member

Comment 3 by sheriffbot@chromium.org, Apr 19 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
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
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 19 2017

Labels: -merge-approved-59 merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a74d0227cdfe9c570717e48439aafb73d42f3635

commit a74d0227cdfe9c570717e48439aafb73d42f3635
Author: Min Qin <qinmin@chromium.org>
Date: Wed Apr 19 16:34:35 2017

Record more download resumption UMAs to analyze the how download is resumed

This CL adds the metrics to study how many resumption is attempted for a download.
This allows us to study the high download interruption counts.
And choose a better throttling strategy.

BUG= 712318 
TBR=dtrainor@chromium.org, isherman@chromium.org

Review-Url: https://codereview.chromium.org/2820193002
Cr-Commit-Position: refs/heads/master@{#465451}
(cherry picked from commit bbe9ad733bbdf501cc63db9b624cfed076d33c32)

Review-Url: https://codereview.chromium.org/2831603003 .
Cr-Commit-Position: refs/branch-heads/3071@{#51}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/a74d0227cdfe9c570717e48439aafb73d42f3635/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java
[modify] https://crrev.com/a74d0227cdfe9c570717e48439aafb73d42f3635/tools/metrics/histograms/histograms.xml

Comment 5 by qin...@chromium.org, Apr 21 2017

Status: Fixed (was: Started)

Sign in to add a comment