1141349 - Disable MSE for Mac for Firefox 37

Status: RESOLVED FIXED

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

Description

[Ralph Giles (:rillian)]

We're not confident with the state of Youtube interactions on Mac with MSE enabled. We should disable it once Youtube confirm they've flipped their side.

Comment 1

[Ralph Giles (:rillian)]

Attachment: Disable MSE on Mac for Firefox 37

Comment 2

[Ralph Giles (:rillian)]

[Tracking Requested - why for this release]:

Comment 3

[Ralph Giles (:rillian)]

Chris, can you let us know when Youtube has updated and it's safe to land this? Target dates are Thursday and Monday morning.

Comment 4

[Jet Villegas (inactive)]

(In reply to Ralph Giles (:rillian) from comment #0)
> We're not confident with the state of Youtube interactions on Mac with MSE
> enabled.

Which bug(s) are you tracking for the mac?

Comment 5

[Ralph Giles (:rillian)]

Comment on attachment 8574941 [details] [diff] [review]
Disable MSE on Mac for Firefox 37

Requesting approval for 37 beta. Again, this shouldn't land until we've confirmed YouTube has changed their behaviour to suit.

Approval Request Comment
[Feature/regressing bug #]: MSE
[User impact if declined]: Worse experience playing YouTube video.
[Describe test coverage new/current, TreeHerder]: N/A.
[Risks and why]: This is a pref flip, restoring us to the behaviour in 36 without the MSE feature. Video will be taking a different codepath. We'll have to keep in eye on bug 1131832.
[String/UUID change made/needed]: None.

Comment 6

[Ralph Giles (:rillian)]

(In reply to Jet Villegas (:jet) from comment #4)

> Which bug(s) are you tracking for the mac?

The main issue was bug 1133633 (performance issues on some systems) and concern about the uplift risk of patches currently landing on m-c to address resource contention through the 'dormant decoder' handling. There's no specific tracking bug for the mac dormant issue, but see bugs 1128381, 1128380, 1139779, and deps.

Comment 7

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

I'm holding Beta approval until Ralph confirms that YouTube is ready for the change.

Comment 8

[Chris Peterson [:cpeterson]]

YouTube says they will try to switch Mac back to Flash video on Thursday March 12, but we should verify the site has switched before pushing a beta with MSE disabled.

Comment 9

[Ralph Giles (:rillian)]

I'm still seeing the html player this morning on a test Mac with 37b4 and flash installed.

Comment 10

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

Given comment 9, we're going to wait to take this fix in Beta 6.

Comment 11

[Ralph Giles (:rillian)]

Same today. I'm on PTO next week. Maja, could you check this Monday morning to see if we can land this in time for beta 6?

Just see if youtube defaults to flash instead of html on a Mac with flash installed for Firefox 37 the way it does for Firefox 36. (Check the right-click menu, for example.)

Comment 12

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

I still see HTML5 video on YouTube this (Monday) morning.

Comment 13

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

(In reply to Lawrence Mandel [:lmandel] (use needinfo) from comment #12)
> I still see HTML5 video on YouTube this (Monday) morning.

Same here: I'm still seeing HTML5 video on 37_0b5 on Mac, using a temporary profile; Flash is installed.

Comment 14

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

I'm now once again seeing Flash content on YouTube in 37 Beta 5 on OS X. Looks like the latest push has defaulted back to Flash.

Comment 15

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

Comment on attachment 8574941 [details] [diff] [review]
Disable MSE on Mac for Firefox 37

Now that YouTube is serving Flash content to Firefox on OS X again, let's disable MSE on OS X in Firefox 37. Beta+

Comment 16

[Ryan VanderMeulen [:RyanVM]]

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