1257100 - Report we can play type 'audio/mp4; codecs="mp4a.40.29"'

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

We can play HE-AACv2, but MEdiaSource.isTypeSupported("audio/mp4; codecs=\"mp4a.40.29\"") return false. Ditto for HTMLMediaElement.canPlayType().

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Report we can play type 'audio/mp4; codecs=mp4a.40.29'

Comment 2

[Jean-Yves Avenard [:jya]]

Comment on attachment 8731077 [details] [diff] [review]
Report we can play type 'audio/mp4; codecs=mp4a.40.29'

Review of attachment 8731077 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/media/test/test_can_play_type_mpeg.html
@@ +75,5 @@
>    check("audio/mp4; codecs=\"mp4a.40.5\"", "probably");
>    check("audio/mp4; codecs=mp4a.40.5", "probably");
>    check("audio/x-m4a; codecs=\"mp4a.40.5\"", "probably");
>    check("audio/x-m4a; codecs=mp4a.40.5", "probably");
> +  

white space, can remove CR too.

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1114b55d0da25b717aa02c7a4d9fc9b0efb53bf1
Bug 1257100 - Report we can play type 'audio/mp4; codecs=mp4a.40.29'. r=jya

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/1114b55d0da2

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

Comment on attachment 8731077 [details] [diff] [review]
Report we can play type 'audio/mp4; codecs=mp4a.40.29'

Approval Request Comment
[Feature/regressing bug #]: MSE video on Facebook
[User impact if declined]: Facebook will have to deploy a work around to get MediaSource video to work in Firefox.
[Describe test coverage new/current, TreeHerder]: we have tests to cover this feature, and the additions being made here.
[Risks and why]: Low, this is just basically adding a new value to our set of values being compared against.
[String/UUID change made/needed]: none.

Comment 6

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8731077 [details] [diff] [review]
Report we can play type 'audio/mp4; codecs=mp4a.40.29'

MSE video on FB, Aurora47+, Beta46+

Comment 7

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/7f0c41c49f0e

Comment 8

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/releases/mozilla-beta/rev/f9d014febe09