1172392 - Align update of audio end time of decoded stream with that of AudioSink

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

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

This will make the behavior of decoded stream more consistent with that of AudioSink and easier to unify them which are both consumers of audio/video data.

Comment 1

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

Try is green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=52b59eeef3ba

Comment 2

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

Attachment: 1172392_align_update_audio_end_time-v1.patch

We should update mAudioEndTime as soon as we push audio data to the stream. This is consistent with the behavior of AudioSink and makes the result of GetDecodedAudioDuration() and AudioDecodedUsecs() more accurate.

Comment 3

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

Try is green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=f1a5c1e85a93

Comment 4

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

Thanks for the review!

Comment 5

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

Attachment: 1172392_align_update_audio_end_time-v2.patch

rebase.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3c1dcd0c1620

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/3c1dcd0c1620