Closed
Bug 1336400
Opened 8 years ago
Closed 7 years ago
Remove HTMLMediaElement::mozAutoplayEnabled
Categories
(Core :: Audio/Video: Playback, defect, P3)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: Ms2ger, Assigned: alwu)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, site-compat)
Attachments
(3 files)
No description provided.
Updated•8 years ago
|
Component: Audio/Video → Audio/Video: Playback
Updated•8 years ago
|
Priority: -- → P3
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → alwu
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Blocks: block-autoplay
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8927784 [details]
Bug 1336400 - part2 : remove MozAutoplayEnabled attribute from HTMLMediaElement.
https://reviewboard.mozilla.org/r/199062/#review204142
::: dom/webidl/HTMLMediaElement.webidl:115
(Diff revision 1)
> [Pref="media.test.dumpDebugInfo"]
> void mozDumpDebugInfo();
>
> attribute MediaStream? srcObject;
>
> attribute boolean mozPreservesPitch;
Not about this bug, but do we have a bug to remove also mozPreservesPitch
Attachment #8927784 -
Flags: review?(bugs) → review+
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8927783 [details]
Bug 1336400 - part1 : remove MozAutoplayEnabled from videocontrols.xml.
https://reviewboard.mozilla.org/r/199060/#review204274
Attachment #8927783 -
Flags: review?(jaws) → review+
Assignee | ||
Comment 6•7 years ago
|
||
(In reply to Olli Pettay [:smaug] from comment #4)
> Not about this bug, but do we have a bug to remove also mozPreservesPitch
Yes, bug1336399.
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8927785 [details]
Bug 1336400 - part3 : move all autoplay checks to CanActivateAutoplay().
https://reviewboard.mozilla.org/r/199078/#review204332
Attachment #8927785 -
Flags: review?(jwwang) → review+
Assignee | ||
Comment 8•7 years ago
|
||
Pushed by alwu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/40241a2e3c1d
part1 : remove MozAutoplayEnabled from videocontrols.xml. r=jaws
https://hg.mozilla.org/integration/autoland/rev/a76b925084ea
part2 : remove MozAutoplayEnabled attribute from HTMLMediaElement. r=smaug
https://hg.mozilla.org/integration/autoland/rev/a4810cd2202e
part3 : move all autoplay checks to CanActivateAutoplay(). r=jwwang
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/40241a2e3c1d
https://hg.mozilla.org/mozilla-central/rev/a76b925084ea
https://hg.mozilla.org/mozilla-central/rev/a4810cd2202e
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Comment 11•7 years ago
|
||
Posted the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2017/htmlmediaelement-mozautoplayenabled-has-been-removed/
Comment 12•7 years ago
|
||
Not documented on MDN; however, the change is now mentioned briefly on Firefox 59 for developers.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•