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

Issue 731667 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 728294



Sign in to add a comment

Record metrics regarding media element play() promise rejection

Project Member Reported by mlamouri@chromium.org, Jun 9 2017

Issue description

This should help figure out how common some rejections are and the impact on developers.
 
Blocking: 728294
Labels: Merge-Request-60
This is a very simple metrics change. Would be great if we could merge to Beta to have results early.

CL: https://chromium-review.googlesource.com/c/528119/
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 20 2017

Labels: -Merge-Request-60 Hotlist-Merge-Review Merge-Review-60
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-60 Merge-Rejected-60
What's the impact of this bug?  It's pretty late to merge this kind of change, so I'd prefer to wait until M61.  If you feel strongly it should be merged you can add the Merge-Request-60 label and we'll re-review.
Cc: bustamante@chromium.org
bustamanto@, I'm not sure you've read my message above based on your reply. This is not a bug fix but metrics. This is simply recording a histogram in four different places. Nothing fancy. Sure, we can wait the next release, but this change is as safe as it gets.
Status: Fixed (was: Started)

Sign in to add a comment