Closed Bug 1108059 Opened 6 years ago Closed 6 years ago

Enable mp4 instead of webm for MediaSource on MacOS X.

Categories

(Core :: Audio/Video, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37
Tracking Status
firefox36 --- fixed
firefox37 --- fixed

People

(Reporter: rillian, Assigned: rillian)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This makes the Mac setting the same as on Windows, which will reduce
the amount of testing we get for WebM, since only Linux will be
left with webm enabled, but reduces the exposed code surface while
we're trying to stabilize things enough to ship support.
Assignee: nobody → giles
Attachment #8532674 - Flags: review?(ajones)
Pushed to inbound based in verbal review from :k17e.

https://hg.mozilla.org/integration/mozilla-inbound/rev/8878e674973d
Depends on: 1098126
Attachment #8532674 - Flags: review?(ajones) → review+
https://hg.mozilla.org/mozilla-central/rev/8878e674973d
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Comment on attachment 8532674 [details] [diff] [review]
Flip MediaSource codecs for MacOS X

Approval Request Comment

[Feature/regressing bug #]: MSE

[User impact if declined]:

Most of YouTube will still work, but not all videos will play without Flash on MacOS at the previous pref setting. Mostly, we want to get as much testing as possible for the mp4 configuration, so aligning with Nightly makes sense.

[Describe test coverage new/current, TBPL]:

We have some tests in tree, but the idea is to get real-world testing against sites like Youtube using MediaSource. Testing the same code on Windows and Mac should make the experience more reliable.

[Risks and why]:

This is associated with MSE only. Youtube may be less stable since the mp4 codepath has seen less testing than webm.

[String/UUID change made/needed]: None.
Attachment #8532674 - Flags: approval-mozilla-aurora?
Attachment #8532674 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.