Closed
Bug 1043696
Opened 10 years ago
Closed 10 years ago
Enable Apple PlatformDecoderModule for HTML5 video playback (MP4 playback on OSX)
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla34
Tracking | Status | |
---|---|---|
relnote-firefox | --- | 35+ |
People
(Reporter: rillian, Assigned: rillian)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-needed)
Attachments
(1 file, 3 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review |
Quoth :cpearce: Now, try exposing the MP4Reader in normal <video> on MacOSX, and see what tests fail. We want H.264/AAC on MacOSX in <video> ASAP. :)
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → giles
Comment 1•10 years ago
|
||
I bet we need bug 1039128 fixed before this is green. kft is working on a fix (drain isn't working properly).
Comment 2•10 years ago
|
||
How does this bug relate to Bug 1030516?
Assignee | ||
Comment 3•10 years ago
|
||
It's probably a duplicate. Marking as blocking until ajones can clarify if he wanted something different there.
Blocks: 1030516
Flags: needinfo?(ajones)
Updated•10 years ago
|
Summary: Enable Apple PlatformDecoderModule for HTML5 video playback → Enable Apple PlatformDecoderModule for HTML5 video playback (MP4 playback on OSX)
Assignee | ||
Comment 6•10 years ago
|
||
Patch pushed to try for osx hosts. Passed on 10.6 as expected, since decoding doesn't work there. Got timeouts and errors on 10.8:
test_autoplay_contentEditable.html | Test timed out.
test_buffered.html | gizmo.mp4: Should be buffered in one range - got 0, expected 1
test_buffered.html | uncaught exception - IndexSizeError: Index or size is negative or greater than the allowed amount at http://mochi.test:8888/tests/content/media/test/test_buffered.html:32
test_bug448534.html | Test timed out.
test_bug465498.html | Test timed out.
test_bug495145.html | Test timed out.
Trying again now that the range calculation cleanup patches have landed.
Assignee | ||
Comment 7•10 years ago
|
||
N.B. If we can get tests passing for osx before we get mp3-in-mp4 support landed, we want to enable this in nightly just for osx. Doing so unconditionally would be a regression on Windows.
Assignee | ||
Comment 8•10 years ago
|
||
Same errors with bug 1050060 fixed. https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=dd5e88f7a193
Comment 9•10 years ago
|
||
W(In reply to Ralph Giles (:rillian) from comment #7)
> N.B. If we can get tests passing for osx before we get mp3-in-mp4 support
> landed, we want to enable this in nightly just for osx. Doing so
> unconditionally would be a regression on Windows.
We can pref off mp3-in-mp4.
Assignee | ||
Comment 10•10 years ago
|
||
New try push since some demuxer changes have landed. https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=a86aa41e92e
Assignee | ||
Comment 11•10 years ago
|
||
We expect to get this working on Mac before other platforms and want to enable it asap since it's the path to mp4 playback there. Adapt to patch to the conditional on the platform.
Attachment #8472735 -
Attachment is obsolete: true
Updated•10 years ago
|
Assignee | ||
Comment 12•10 years ago
|
||
Comment on attachment 8476982 [details] [diff] [review]
Enable the fmp4 decoder for <video> on non-release osx builds
Review of attachment 8476982 [details] [diff] [review]:
-----------------------------------------------------------------
Asking for review now, since we're nearly ready to push the switch. Will land once I clean try pushes.
Attachment #8476982 -
Flags: review?(cpearce)
Comment 13•10 years ago
|
||
Comment on attachment 8476982 [details] [diff] [review]
Enable the fmp4 decoder for <video> on non-release osx builds
Review of attachment 8476982 [details] [diff] [review]:
-----------------------------------------------------------------
::: modules/libpref/init/all.js
@@ +245,4 @@
> #ifdef MOZ_FMP4
> pref("media.fragmented-mp4.enabled", true);
> pref("media.fragmented-mp4.ffmpeg.enabled", false);
> +#if defined(XP_MACOSX) && !defined(RELEASE_BUILD)
Remove the !defined(RELEASE_BUILD) check. We can pref it off if we can't stabilize it by the time this gets to beta.
Attachment #8476982 -
Flags: review?(cpearce) → review+
Assignee | ||
Comment 14•10 years ago
|
||
Update addressing review comments and changes from bug 1057879. Carrying forward r=cpearce
Attachment #8476982 -
Attachment is obsolete: true
Assignee | ||
Comment 15•10 years ago
|
||
Work around bug 1058897. Carrying forward r=cpearce again.
Attachment #8479321 -
Attachment is obsolete: true
Assignee | ||
Comment 16•10 years ago
|
||
Green on try https://tbpl.mozilla.org/?tree=Try&rev=7232395497f6
Assignee | ||
Comment 17•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Comment 19•10 years ago
|
||
Yes! Thanks to all who made this possible!
Any chance this and any necessary fixes can be uplifted to Fx 33 / Aurora?
This is Lion and newer, right?
Release Note Request (optional, but appreciated)
[Why is this notable]: First time h264 is available on Mac without plugins
[Suggested wording]: (Added) Built-in support for H264 (MP4) on OS X Lion(??) and newer through native APIs.
[Links (documentation, blog post, etc)]: -
relnote-firefox:
--- → ?
Comment 20•10 years ago
|
||
(In reply to Florian Bender from comment #19)
> Any chance this and any necessary fixes can be uplifted to Fx 33 / Aurora?
There's Buckley's chance of that happening. There are way too many commits to the MP4 parser and related code that this depends on.
Anthony
Comment 21•10 years ago
|
||
Yeah I thought so. Thanks anyway!
Comment 22•10 years ago
|
||
With `media.apple.mp4.enabled` set to true, the video on youtube HTML5's player becomes black/blank, the audio for me. A quick fix is to manually toggle that flag to false.
I am using a Late-2009 MacBook running Yosemite public beta 2.
Updated•10 years ago
|
Keywords: dev-doc-needed
Comment 23•10 years ago
|
||
(In reply to Jonny Barnes from comment #22)
> With `media.apple.mp4.enabled` set to true, the video on youtube HTML5's
> player becomes black/blank, the audio for me. A quick fix is to manually
> toggle that flag to false.
>
> I am using a Late-2009 MacBook running Yosemite public beta 2.
This was tracked in bug 1054789, and is thought to be fixed. Can you re-test nightly, and if it still occurs, please file a new bug. Thanks!
Comment 24•10 years ago
|
||
(In reply to Chris Pearce (:cpearce) from comment #23)
> (In reply to Jonny Barnes from comment #22)
> > With `media.apple.mp4.enabled` set to true, the video on youtube HTML5's
> > player becomes black/blank, the audio for me. A quick fix is to manually
> > toggle that flag to false.
> >
> > I am using a Late-2009 MacBook running Yosemite public beta 2.
>
> This was tracked in bug 1054789, and is thought to be fixed. Can you re-test
> nightly, and if it still occurs, please file a new bug. Thanks!
Nightly works with YouTube as expected now.
Comment 25•10 years ago
|
||
Added to the release notes with "Built-in support for H264 (MP4) on Mac OS X Lion and newer through native APIs" as wording.
Comment 26•10 years ago
|
||
Also supported on Snow Leopard (10.6). See bug 1070703.
Comment 27•10 years ago
|
||
OK, Updated! thanks
Comment 29•10 years ago
|
||
MP4 support on Mac OSX has been pushed out to Firefox 35 due to unresolved issues in the demuxer.
Depends on: 1100482
Sylvestre or lmandel, the release notes may need updating here.
Flags: needinfo?(sledru)
Flags: needinfo?(lmandel)
Comment 31•10 years ago
|
||
OK. Thanks. Moved to 35 then!
Comment 32•10 years ago
|
||
(In reply to Anthony Jones (:kentuckyfriedtakahe, :k17e) from comment #29)
> MP4 support on Mac OSX has been pushed out to Firefox 35 due to unresolved
> issues in the demuxer.
Do you have a link to those issues so that I can follow them?
This is pretty disappointing for us :(
You need to log in
before you can comment on or make changes to this bug.
Description
•