1379056 - [Fennec][HLS] GeckoHlsPlayer should handle the case when obtain a negative duration from ExoPlayer.

Status: RESOLVED FIXED

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

Description

[Kilik Kuo [:kikuo] (inactive)]

Follow-up of Bug 1378852 Comment 3.

Comment 1

[Kilik Kuo [:kikuo] (inactive)]

Attachment: Bug 1379056 - GeckoHlsPlayer should handle the case when obtain a negative duration from ExoPlayer.

We might get a negative duration from ExoPlayer if it's unknown at that time,
Setting it to 0 to align the default duration value in VideoInfo/AudioInfo.

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

Comment 2

[James Cheng[:JamesCheng]]

Comment on attachment 8884170 [details]
Bug 1379056 - GeckoHlsPlayer should handle the case when obtain a negative duration from ExoPlayer.

https://reviewboard.mozilla.org/r/155090/#review160156

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e8851a94e09f
GeckoHlsPlayer should handle the case when obtain a negative duration from ExoPlayer. r=JamesCheng

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/e8851a94e09f