Closed Bug 1232313 Opened 5 years ago Closed 5 years ago

Autophone - Mdm failure - dom/media/test/test_unseekable.html | Test timed out

Categories

(Firefox for Android :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox47 --- fixed
firefox48 --- fixed
fennec + ---

People

(Reporter: bc, Assigned: jya)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

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?
Flags: needinfo?(jyavenard)
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.
Flags: needinfo?(jyavenard)
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.
Flags: needinfo?(jyavenard)
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
Flags: needinfo?(jyavenard)
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+
https://hg.mozilla.org/mozilla-central/rev/0ed310b4f4d0
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
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.
Blocks: 1269325
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+
You need to log in before you can comment on or make changes to this bug.