Closed
Bug 1377839
Opened 8 years ago
Closed 8 years ago
Categories
(Core :: Audio/Video: Playback, enhancement)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
INVALID
People
(Reporter: ayang, Assigned: ayang)
References
Details
(Keywords: compat)
https://github.com/webcompat/web-bugs/issues/7245
Chrome can play it.
|
Assignee | |
Comment 1•8 years ago
|
||
No 'ftyp' box, reject at mp4_demuxer::MoofParser::Metadata() ftyp/moov parsing.
See Also: → 1336271
|
|
Assignee | |
Comment 2•8 years ago
|
||
One way to solve this is to add 'ftyp' byte range only if 'ftyp' is exist and its byte range is smaller than 'moov'. And only using 'moov' as the necessary condition here.
I think it may also fix bug 1336271.
|
|
Assignee | |
Comment 3•8 years ago
|
||
Btw, rust parser doesn't need 'ftyp'.
Stagefright fails to parse stream if 'ftyp' is not at the beginning.
|
||
Comment 4•8 years ago
|
||
The file are grossly invalid.
We shouldn't support them IMHO.
Adding that they wouldn't open even if the demuxer supported them as we have no way of sniffing such file and detect they are mp4.
(per spec, mp4 sniffing is based on the ftyp content)
|
|
Assignee | |
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
|
|
Assignee | |
Comment 5•8 years ago
|
||
Another webcompat bug without 'ftyp'.
https://github.com/webcompat/web-bugs/issues/7121
|
||
Updated•8 years ago
|
See Also: → https://webcompat.com/issues/7121
|
|
||
Updated•8 years ago
|
See Also: → https://webcompat.com/issues/7245
You need to log in
before you can comment on or make changes to this bug.
Description
•