1070703 - Add mp4 support in 10.6 and 10.7 on Aurora

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Jean-Yves Avenard [:jya]]

If we enable VideoToolbox in 10.6 or 10.7, no video appears.

VideoToolbox by default output BGRA32 type buffer.

Comment 1

[Jean-Yves Avenard [:jya]]

Attachment: Set output format for VideoToolbox decoder

Set output format

Comment 2

[Jean-Yves Avenard [:jya]]

This patch is only required if we don't want to backport all the changes related to using IOSurface (bug 1059066 and bug 1064847).

If we are going to, this can be closed as WON'T FIX

Comment 3

[Jean-Yves Avenard [:jya]]

Note that "FullRangeVideo", a.k.a PC Levels format isn't supported in 10.6 and 10.7, only in 10.8 and later.
Hence why I set the format to kCVPixelFormatType_420YpCbCr8BiPlanarVideoRange rather than kCVPixelFormatType_420YpCbCr8BiPlanarFullRange

Comment 4

[Ralph Giles (:rillian)]

Comment on attachment 8492734 [details] [diff] [review]
Set output format for VideoToolbox decoder

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

I'm confused. Does comment 2 mean this patch is intended for Aurora only? If so please state that explicitly. By default patches are assumed to apply to m-c.

Please state _why_ a change in being made in the commit message body.

Comment 5

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

The patch is intended for Aurora only and it is simply the last part of https://bug1059066.bugzilla.mozilla.org/attachment.cgi?id=8485498

The intention is to backport as little as possible to get 10.6 and 10.7 working with software decoders in 34.

Comment 6

[Jean-Yves Avenard [:jya]]

Attachment: 1070703_Enable_mp4_in_106_107.patch

Patch for aurora

Comment 7

[Jean-Yves Avenard [:jya]]

Attachment: 1070703_Enable_mp4_in_106_107.patch

Update mochi test from bug 1067697

Comment 8

[Ralph Giles (:rillian)]

Comment on attachment 8494269 [details] [diff] [review]
1070703_Enable_mp4_in_106_107.patch

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

Thanks for the roll-up patch.

Comment 9

[Ralph Giles (:rillian)]

Comment on attachment 8494269 [details] [diff] [review]
1070703_Enable_mp4_in_106_107.patch

Approval Request Comment
[Feature/regressing bug #]: 941296

[User impact if declined]: 

HTML <video> playback of mp4 files won't be available to users on MacOS X 10.6 and 10.7, complicating deployment for web developers.

[Describe test coverage new/current, TBPL]:

This is a backport of similar code in m-c. Support on 10.6 is tested by mochitests on tbpl.

[Risks and why]: 

We believe risk is low. This is a new feature in 34, so there's no possibility of regression. The code changes to enable this on older versions of MacOS are straightforward: explicitly requesting what is the default output format on 10.8+, and checking additional paths when loading platform frameworks.

[String/UUID change made/needed]: None

Comment 10

[Jean-Yves Avenard [:jya]]

It requires the bump stack size patch to prevent occasional crashes in 10.6

Comment 11

[Jean-Yves Avenard [:jya]]

https://tbpl.mozilla.org/?tree=Try&rev=e45eea7a4f9d

Comment 12

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8494269 [details] [diff] [review]
1070703_Enable_mp4_in_106_107.patch

I like my Web platform consistent. Thank you for the simplified patch for uplift. Aurora+

Comment 13

[Ralph Giles (:rillian)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c8ca7d751f8d

Comment 14

[Jean-Yves Avenard [:jya]]

shouldn't this bug be closed as 34 has the code in ...

Comment 15

[Ralph Giles (:rillian)]

I think so too. I suspect it wasn't closed because the sheriff scripts close bugs based on m-c merge, and that isn't part of this bug.