Move setting of AudioFrame::timestamp_ into NetEq

Move setting of AudioFrame::timestamp_ into NetEq

This was previously done in AcmReceiver, but belongs in NetEq where the
rest of the AudioFrame fields are populated.

BUG=webrtc:5669, webrtc:5607
Committed: https://crrev.com/15c51e355f9ade7fbeb7f7ac6784c0f24446ab7e
Cr-Commit-Position: refs/heads/master@{#12265}

[hlundin-webrtc, 2016-04-06 09:30:48]

Patchset #1 (id:1) has been deleted

[hlundin-webrtc, 2016-04-06 09:34:06]

henrik.lundin@webrtc.org changed reviewers:
+ minyue@webrtc.org

[hlundin-webrtc, 2016-04-06 09:34:07]

https://codereview.webrtc.org/1863993002/diff/20001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/neteq/include/neteq.h (right):

https://codereview.webrtc.org/1863993002/diff/20001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/neteq/include/neteq.h:249: virtual
rtc::Optional<uint32_t> GetPlayoutTimestamp() const = 0;
This was always essentially a const method, but it was not declared as such
before.

[minyue-webrtc, 2016-04-06 11:16:26]

lgtm

https://codereview.webrtc.org/1863993002/diff/20001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/neteq/neteq_impl.cc (right):

https://codereview.webrtc.org/1863993002/diff/20001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/neteq/neteq_impl.cc:973: audio_frame->timestamp_ =
maybe too many lines here

[hlundin-webrtc, 2016-04-06 11:18:21]

Thanks!

https://codereview.webrtc.org/1863993002/diff/20001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/neteq/neteq_impl.cc (right):

https://codereview.webrtc.org/1863993002/diff/20001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/neteq/neteq_impl.cc:973: audio_frame->timestamp_ =
On 2016/04/06 11:16:26, minyue-webrtc wrote:
> maybe too many lines here

This is what git cl format gave me, and I'd rather not fight it today. :)

[hlundin-webrtc, 2016-04-06 11:19:59]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2016-04-06 11:20:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1863993002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1863993002/20001

[commit-bot: I haz the power, 2016-04-06 13:20:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-06 13:20:51]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[hlundin-webrtc, 2016-04-06 15:33:32]

The CQ bit was checked by henrik.lundin@webrtc.org

[commit-bot: I haz the power, 2016-04-06 15:33:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1863993002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1863993002/20001

[commit-bot: I haz the power, 2016-04-06 15:39:01]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-04-06 15:39:05]

Description was changed from

==========
Move setting of AudioFrame::timestamp_ into NetEq

This was previously done in AcmReceiver, but belongs in NetEq where the
rest of the AudioFrame fields are populated.

BUG=webrtc:5669,webrtc:5607
==========

to

==========
Move setting of AudioFrame::timestamp_ into NetEq

This was previously done in AcmReceiver, but belongs in NetEq where the
rest of the AudioFrame fields are populated.

BUG=webrtc:5669,webrtc:5607

Committed: https://crrev.com/15c51e355f9ade7fbeb7f7ac6784c0f24446ab7e
Cr-Commit-Position: refs/heads/master@{#12265}
==========

[commit-bot: I haz the power, 2016-04-06 15:39:06]

Patchset 1 (id:??) landed as
https://crrev.com/15c51e355f9ade7fbeb7f7ac6784c0f24446ab7e
Cr-Commit-Position: refs/heads/master@{#12265}