New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 775613 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug

Blocking:
issue 747563


Participants' hotlists:
Downloads-Foreground-Service


Sign in to add a comment

[Downloads] UMA for tracking downloads notifications

Project Member Reported by jming@chromium.org, Oct 17 2017

Issue description

There is a larger refactor to make downloads a foreground service on all versions of Android. During the refactor, a few tradeoffs had to be made (ie. optimizing for single download or multiple). Therefore, more metrics need to be taken to understand how to make these tradeoffs in the future, specifically around notification interactions, number of notifications, service lifecycle, unexpected/expected service stops, etc.

Currently, the code for the refactor lives behind a flag chrome://flags (Enable downloads foreground) and will likely be activated in M64. However, it would be helpful to understand the status quo in terms of metrics around these areas in M63.
 

Comment 1 by jming@chromium.org, Oct 17 2017

Status: Fixed (was: Assigned)
Fixed in: https://chromium-review.googlesource.com/c/chromium/src/+/710579, merge would be helpful.
Project Member

Comment 2 by sheriffbot@chromium.org, Oct 18 2017

Labels: -Merge-Request-63 Hotlist-Merge-Approved Merge-Approved-63
Your change meets the bar and is auto-approved for M63. Please go ahead and merge the CL to branch 3239 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 3 by sheriffbot@chromium.org, Oct 23 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by sheriffbot@chromium.org, Oct 27 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by jming@chromium.org, Oct 27 2017

Labels: -Hotlist-Merge-Approved -Merge-Approved-63

Sign in to add a comment