1580597 - Set correct timestamps for WebRTCMediaDataEncoderCodec.

Status: RESOLVED FIXED

(Core :: Audio/Video, defect, P2)

Description

[John Lin [:jhlin][:jolin]]

AppleVTEncoder doesn't set the timestamp value in VideoData and AndroidDataEncoder should send timestamp in microseconds to CodecProxy.

Comment 1

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1580597 - p1: pass timestamp values correctly. r?jya

Bug 1568101/D40532 captures the input timestamp and uses it in the resolve
function. However, there could be multiple encoded frames returned (e.g.,
for Drain()), and all of them will get the same captured timestamp. Read
back the timestamp from encoded data makes sure every frame gets the right
value. Also convert it for EncodedImage::_timeStamp uses different unit.

Depends on D45773

Comment 2

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1580597 - p2: set timestamp and duration of the encoded frame. r?jya

Also remove the definition of unused struct for tracking of input timestamp.

Depends on D45774

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cbcba8e0c801
p1: pass timestamp values correctly. r=jya
https://hg.mozilla.org/integration/autoland/rev/ce274dd6db3a
p2: set timestamp and duration of the encoded frame. r=jya

Comment 4

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/cbcba8e0c801
https://hg.mozilla.org/mozilla-central/rev/ce274dd6db3a