Configure VoE NACK through AudioReceiveStream::Config.

Configure VoE NACK through AudioReceiveStream::Config, for receive streams. Also minor refactoring of WVoE unit test.

BUG=webrtc:4690
Committed: https://crrev.com/8189b02feaae73e542190f79c57e858fd7d84f3d
Cr-Commit-Position: refs/heads/master@{#13140}

[the sun, 2016-06-13 10:05:21]

Description was changed from

==========
Configure VoE NACK through AudioReceiveStream::Config.

BUG=webrtc:4690
==========

to

==========
Configure VoE NACK through AudioReceiveStream::Config, for receive streams. Also
minor refactoring of WVoE unit test.

BUG=webrtc:4690
==========

[the sun, 2016-06-13 10:05:34]

solenberg@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[kwiberg-webrtc, 2016-06-13 13:47:47]

lgtm

https://codereview.webrtc.org/2060813002/diff/1/webrtc/media/engine/webrtcvoi...
File webrtc/media/engine/webrtcvoiceengine_unittest.cc (right):

https://codereview.webrtc.org/2060813002/diff/1/webrtc/media/engine/webrtcvoi...
webrtc/media/engine/webrtcvoiceengine_unittest.cc:136: }
Nice simplification, but it makes the diff harder to read. A.k.a., please do
this sort of thing in a separate CL next time. :-)

[the sun, 2016-06-13 16:55:38]

https://codereview.webrtc.org/2060813002/diff/1/webrtc/media/engine/webrtcvoi...
File webrtc/media/engine/webrtcvoiceengine_unittest.cc (right):

https://codereview.webrtc.org/2060813002/diff/1/webrtc/media/engine/webrtcvoi...
webrtc/media/engine/webrtcvoiceengine_unittest.cc:136: }
On 2016/06/13 13:47:47, kwiberg-webrtc wrote:
> Nice simplification, but it makes the diff harder to read. A.k.a., please do
> this sort of thing in a separate CL next time. :-)

Acknowledged.

[the sun, 2016-06-14 16:04:48]

The CQ bit was checked by solenberg@webrtc.org

[the sun, 2016-06-14 16:04:48]

The patchset sent to the CQ was uploaded after l-g-t-m from kwiberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2060813002/#ps60001 (title:
"rebase")

[commit-bot: I haz the power, 2016-06-14 16:04:52]

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

[commit-bot: I haz the power, 2016-06-14 16:04:53]

This CL has an open dependency (Issue 1955363003 Patch 120001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[the sun, 2016-06-14 17:48:46]

The CQ bit was checked by solenberg@webrtc.org

[the sun, 2016-06-14 17:48:47]

The patchset sent to the CQ was uploaded after l-g-t-m from kwiberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2060813002/#ps100001 (title:
"format")

[commit-bot: I haz the power, 2016-06-14 17:48:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2060813002/100001

[commit-bot: I haz the power, 2016-06-14 19:13:03]

Description was changed from

==========
Configure VoE NACK through AudioReceiveStream::Config, for receive streams. Also
minor refactoring of WVoE unit test.

BUG=webrtc:4690
==========

to

==========
Configure VoE NACK through AudioReceiveStream::Config, for receive streams. Also
minor refactoring of WVoE unit test.

BUG=webrtc:4690
==========

[commit-bot: I haz the power, 2016-06-14 19:13:05]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-06-14 19:13:16]

Description was changed from

==========
Configure VoE NACK through AudioReceiveStream::Config, for receive streams. Also
minor refactoring of WVoE unit test.

BUG=webrtc:4690
==========

to

==========
Configure VoE NACK through AudioReceiveStream::Config, for receive streams. Also
minor refactoring of WVoE unit test.

BUG=webrtc:4690

Committed: https://crrev.com/8189b02feaae73e542190f79c57e858fd7d84f3d
Cr-Commit-Position: refs/heads/master@{#13140}
==========

[commit-bot: I haz the power, 2016-06-14 19:13:17]

Patchset 6 (id:??) landed as
https://crrev.com/8189b02feaae73e542190f79c57e858fd7d84f3d
Cr-Commit-Position: refs/heads/master@{#13140}