1364296 - Enable dom/media/test/test_unseekable.html on Android

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Audio/Video, enhancement)

Comment 1

[Alastor Wu [:alwu]]

Hi, Alfredo,
I saw that you added "street.mp4" to gUnseekableTests in bug959861, but I opened it with firefox and it seems seekable. Why you added this file to gUnseekableTests?
Thanks!

Comment 2

[Alastor Wu [:alwu]]

In addition, it seems that we can't read the correct duration of that file. 
When I dump the value from MediaDecoder::GetDuration(), it would return "inf". But if you open that file with Quicktime player, it would show the duration is 8 seconds.

Comment 3

[Alfredo Yang (:alfredo)]

Bug959861 is for b2g. Fragmented mp4 used to be unseekable on firefox iirc.

Comment 4

[Alastor Wu [:alwu]]

Attachment: Bug 1364296 - enable dom/media/test/test_unseekable.html on Android.

Now the fragent mp4 is seekable, so we should remove "street.mp4" from unseekable test.

Review commit: https://reviewboard.mozilla.org/r/139214/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/139214/

Comment 5

[Alastor Wu [:alwu]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=24b2e448e6aef19da3067620b7b353654c2273b8

Comment 6

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8867634 [details]
Bug 1364296 - enable dom/media/test/test_unseekable.html on Android.

https://reviewboard.mozilla.org/r/139214/#review142852

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5e10a4052ff8
enable dom/media/test/test_unseekable.html on Android. r=jwwang

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5e10a4052ff8