Closed
Bug 1477011
Opened 6 years ago
Closed 6 years ago
canPlayType(video/mp4; codecs="vp9") returns ""
Categories
(Core :: Audio/Video: Playback, enhancement)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: jya, Assigned: jya)
References
Details
Attachments
(1 file)
VP9 in mp4 is supported, yet canPlayType returns an empty string here.
Assignee | ||
Comment 1•6 years ago
|
||
MediaSource.isTypeSupported("video/mp4; codecs="vp9") returns true however...
Assignee | ||
Comment 2•6 years ago
|
||
This is due to https://searchfox.org/mozilla-central/source/dom/html/HTMLMediaElement.cpp#4803 // We don't have a demuxer that can handle VP9 in non-fragmented MP4. // So special-case VP9 in MP4 here, as we assume canPlayType() implies // non-fragmented MP4 anyway. Note we report that we can play VP9 // in MP4 in MediaSource.isTypeSupported(), as the fragmented MP4 // demuxer can handle VP9 in fragmented MP4. I don't think that assumption is still true now that we've switched to the rust mp4 demuxer
See Also: → 1338032
Assignee | ||
Comment 3•6 years ago
|
||
Crafted https://raw.githubusercontent.com/jyavenard/htmltests/master/mediatest/mp4/bipbop_vp9.mp4 that shows that it plays. Should remove that restriction...
Assignee | ||
Comment 4•6 years ago
|
||
Since we switched to the mp4 rust demuxer, VP9 in mp4 is always supported.
Comment 5•6 years ago
|
||
Comment on attachment 8993591 [details] Bug 1477011 - Report vp9 in mp4 as supported. r?dminor Dan Minor [:dminor] has approved the revision. https://phabricator.services.mozilla.com/D2261
Attachment #8993591 -
Flags: review+
Pushed by jyavenard@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/afcfe1cd7c55 Report vp9 in mp4 as supported. r=dminor
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/afcfe1cd7c55
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Updated•6 years ago
|
Assignee: nobody → jyavenard
You need to log in
before you can comment on or make changes to this bug.
Description
•