Closed
Bug 839055
Opened 12 years ago
Closed 12 years ago
H.264/AAC support: abort playback of resource if the video or audio stream is unplayable
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: epinal99-bugzilla2, Assigned: cpearce)
References
()
Details
Attachments
(1 file)
(deleted),
patch
|
padenot
:
review+
|
Details | Diff | Splinter Review |
STR:
1) Start the latest nightly with support of H.264/AAC in MP4
2) Open this trailer http://www.tools4movies.com/dvd_catalyst_profile_samples/Harold%20Kumar%203%20Christmas%20bionic%20fast.mp4
Result: audio but no video (Win 7).
MediaInfo log:
General
Complete name : Harold Kumar 3 Christmas bionic fast.mp4
Format : MPEG-4
Format profile : Base Media
Codec ID : isom
File size : 20.3 MiB
Duration : 2mn 28s
Overall bit rate mode : Variable
Overall bit rate : 1 143 Kbps
Movie name : Harold Kumar 3 Christmas
Writing application : Lavf52.103.0
Cover : Yes
Video
ID : 1
Format : MPEG-4 Visual
Format profile : Simple@L1
Format settings, BVOP : No
Format settings, QPel : No
Format settings, GMC : No warppoints
Format settings, Matrix : Default (H.263)
Codec ID : 20
Duration : 2mn 28s
Bit rate : 1 006 Kbps
Width : 856 pixels
Height : 368 pixels
Display aspect ratio : 2.35:1
Frame rate mode : Constant
Frame rate : 23.976 fps
Color space : YUV
Chroma subsampling : 4:2:0
Bit depth : 8 bits
Scan type : Progressive
Compression mode : Lossy
Bits/(Pixel*Frame) : 0.133
Stream size : 17.8 MiB (88%)
Writing library : Lavc52.115.0
Audio
ID : 2
Format : AAC
Format/Info : Advanced Audio Codec
Format profile : LC
Codec ID : 40
Duration : 2mn 28s
Bit rate mode : Variable
Bit rate : 128 Kbps
Channel(s) : 2 channels
Channel positions : Front: L R
Sampling rate : 44.1 KHz
Compression mode : Lossy
Stream size : 2.26 MiB (11%)
Assignee | ||
Comment 1•12 years ago
|
||
WMF identifies the video as MFVideoFormat_M4S2, "MPEG-4 part 2"
http://msdn.microsoft.com/en-us/library/windows/desktop/dd757766%28v=vs.85%29.aspx
He whitelist codecs, and H.264 is the only codec we've whitelisted, so we're rejecting this file's video stream as having an unsupported codec.
Chrome and IE can't play this file, so I don't see why we should. However Chrome/IE refuse to play the file entirely, whereas we still play the audio track. Maybe we should change to match Chrome/IE's behaviour and refuse to play a file which contains any unsupported streams...
Blocks: 799315
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to y-ii from comment #2)
> if we can support more, why not?we need to do better
If other browsers don't support the format, there's not much point in us supporting it, as websites relying on this format won't work in other browsers.
Assignee | ||
Comment 4•12 years ago
|
||
As per the discussion here:
https://groups.google.com/forum/?fromgroups=#!topic/mozilla.dev.media/MwnP1YlJpYU
I'll refuse to play videos which have an audio or video stream which fail to load in the WMFReader.
Assignee | ||
Comment 5•12 years ago
|
||
Abort playback of resource if the video or audio stream is unplayable.
Chris, will your patch abort playback of h.264 video with unsupported AC3 audio codec (FF21 reads the video but not the audio) like http://www.auby.no/files/video_tests/h264_720p_hp_5.1_6mbps_ac3_planet.mp4 ?
Assignee | ||
Comment 7•12 years ago
|
||
Yes, the patch will make Firefox refuse to play files with unsupported audio. IE and Chrome have the same behaviour.
Updated•12 years ago
|
Attachment #712766 -
Flags: review?(paul) → review+
Summary: H.264/AAC support: audio but no video → H.264/AAC support: abort playback of resource if the video or audio stream is unplayable
Assignee | ||
Comment 8•12 years ago
|
||
Comment 9•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in
before you can comment on or make changes to this bug.
Description
•