Log to web console when autoplay blocked

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
a year ago
9 months ago

People

(Reporter: cpearce, Assigned: alwu)

Tracking

(Blocks 1 bug)

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

We should really log to the web console when we block autoplay, in order to help web developers debug their sites.
Reporter

Updated

9 months ago
Assignee: cpearce → nobody
Assignee

Updated

9 months ago
Assignee: nobody → alwu
Assignee

Comment 1

9 months ago
Log to the web console when we block autoplay, in order to help web developers debug their sites.
Comment on attachment 9004419 [details]
Bug 1453862 - Log to web console when autoplay blocked.

Andrea Marchesini [:baku] has approved the revision.
Attachment #9004419 - Flags: review+
In `dom/html/HTMLMediaElement.cpp` on line 4132 there is a typo. It should read `promises`. Perhaps you can fix it with this push?
Flags: needinfo?(alwu)
Assignee

Comment 4

9 months ago
Sure.
Flags: needinfo?(alwu)
Assignee

Comment 5

9 months ago
Because I'm not sure whether Phabricator would send notification when I mention someone's name, add NI just in case.
Flags: needinfo?(francesco.lodolo)
Yep, I receive notifications via phabricator. I think it's up to you to decide if you want to have someone looking at the copy, maybe it's not needed since it's a console error. In case, I put an alternative in the comment.
Flags: needinfo?(francesco.lodolo)
Comment on attachment 9004419 [details]
Bug 1453862 - Log to web console when autoplay blocked.

Francesco Lodolo [:flod] has approved the revision.
Attachment #9004419 - Flags: review+

Comment 8

9 months ago
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9993da7e3300
Log to web console when autoplay blocked. r=flod,baku

Comment 9

9 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9993da7e3300
Status: NEW → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.