Closed Bug 1232313 Opened 6 years ago Closed 5 years ago
Autophone - Mdm failure - dom/media/test/test
_unseekable .html | Test timed out
https://treeherder.mozilla.org/#/jobs?repo=try&revision=a99d064ef982&exclusion_profile=false&selectedJob=14568920 Nexus 5 Android 4.4 >TEST-TIMEOUT | dom/media/test/test_unseekable.html | Test timed out.
Assignee: nobody → esawin
tracking-fennec: ? → +
This looks like an actual bug, we fail playing dom/media/test/street.mp4 on Android (tested on Nexus 7 Android 6.0.1). I've moved the file to http://me73.com/media/street.mp4 for testing. jya, are there known issues with fragmented MP4 playback on modern Android or is it something specific about this file we don't like?
the mp4 demuxer is identical across all platforms, we use our own. So, no, I'm not aware of any issues there when it comes to the demuxer.
I've added a test page which is supposed to auto-play street.mp4, and it fails in the same way on Windows and Linux desktop (see log in comment 1): http://me73.com/media/frag.html. Manually starting playback seems to work (though it's hard to tell since the video is too short to display anything). jya, this seems to be a general demuxer issue we have with auto-playing street.mp4.
yes, this file is missing tfdt box in the moof. ISO 14496-12 states that this box is not compulsory and is only used to determine the time of the first sample in this fragment. otherwise you have to use the total duration + time of all the samples prior... I'll implement that ASAP.
Assignee: esawin → jyavenard
Per ISO 14496-12, the Track fragment decode time (tfdt) is optional. Review commit: https://reviewboard.mozilla.org/r/40357/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/40357/
Attachment #8731105 - Flags: review?(ajones)
Comment on attachment 8731105 [details] MozReview Request: Bug 1232313: [mp4] Use cumulative decode time if tfdt box is not present. r=kentuckyfriedtakahe https://reviewboard.mozilla.org/r/40357/#review36893
Attachment #8731105 - Flags: review?(ajones) → review+
Comment on attachment 8731105 [details] MozReview Request: Bug 1232313: [mp4] Use cumulative decode time if tfdt box is not present. r=kentuckyfriedtakahe Approval Request Comment [Feature/regressing bug #]: 1232313 [User impact if declined]: some files won't play. Blocking bug 1269325 which has been beta+ [Describe test coverage new/current, TreeHerder]: Committed over 2 months ago, in 48 already [Risks and why]: Very low. just going per spec. [String/UUID change made/needed]: none
Attachment #8731105 - Flags: approval-mozilla-beta?
bug 1269325 was approved for uplift, but it relies on this change to be in first.
Comment on attachment 8731105 [details] MozReview Request: Bug 1232313: [mp4] Use cumulative decode time if tfdt box is not present. r=kentuckyfriedtakahe Media playback issues, Beta47+
Attachment #8731105 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.