1490877 - Support final AV1 MP4 MIME type

Status: RESOLVED FIXED

(Firefox :: File Handling, enhancement)

Description

[strobe]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.92 Safari/537.36

Steps to reproduce:

The final AV1 codec string in the MP4 MIME type is not supported in Firefox. Instead the beta AV1 codec string is used.

MediaSource.isTypeSupported('video/mp4; codecs="av01.0.05M.08")
MediaSource.isTypeSupported('video/mp4; codecs="av1")



Actual results:

-> false
-> true


Expected results:

-> true
-> false

Comment 1

[Thomas Daede [:TD-Linux]]

Indeed, it's actually supposed to accept both strings. Will take a look asap.

Comment 2

[Thomas Daede [:TD-Linux]]

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=835b366ab8cb37f73be4bebf802e2ea3bcf553d6

Comment 3

[Thomas Daede [:TD-Linux]]

Attachment: Bug 1490877: Make codecs=av01* mimetypes work with MSE. r=jya

While the MP4 parser correctly handles the av01* codec string, it
is then converted to a video/av01* mimetype to search for a PDM.
The libaom PDM only accepts video/av1, so always produce a video/av1
MIME type from a codecs=av01 string.

Alternately, the AOMDecoder could be made to accept video/av01*, but
this patch matches what VPXDecoder does.

Comment 4

[Jean-Yves Avenard [:jya]]

Comment on attachment 9008636 [details]
Bug 1490877: Make codecs=av01* mimetypes work with MSE. r=jya

Jean-Yves Avenard [:jya] has approved the revision.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2c48f0cea901
Make codecs=av01* mimetypes work with MSE. r=jya

Comment 6

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/2c48f0cea901

Comment 7

[Thomas Daede [:TD-Linux]]

Comment on attachment 9008636 [details]
Bug 1490877: Make codecs=av01* mimetypes work with MSE. r=jya

Approval Request Comment
[Feature/Bug causing the regression]: 1452683
[User impact if declined]: Youtube will not detect AV1 support
[Is this code covered by automated tests?]: Yes
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: No
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: it only affects a feature behind a pref
[String changes made/needed]: None

Comment 8

[Pascal Chevrel:pascalc]

Comment on attachment 9008636 [details]
Bug 1490877: Make codecs=av01* mimetypes work with MSE. r=jya

Low-risk patch improving AV1 support,  uplift approved for 63 beta 7, thanks.

Comment 9

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/releases/mozilla-beta/rev/6ad54436118a