1546506 - Add an assertion to ensure that we won't construct SeekTarget with invalid time

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, task, P2)

Description

[Alastor Wu [:alwu]]

Fork from [1], we would like to see whether the crash in bug1540744 is caused by using invalid time to initialize a SeekTarget.

[1] https://phabricator.services.mozilla.com/D27136#835472

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1546506 - Add an assertion to ensure that we won't construct SeekTarget with invalid time.

We should only use valid time to construct a SeekTarget, because we can't get the value from the invalid time.

This can help us to prevent having a crash when request data value from an invalid time.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f159c693907a
Add an assertion to ensure that we won't construct SeekTarget with invalid time. r=bryce

Comment 3

[Cristian Brindusan [:cbrindusan]]

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