Configure FlexFEC in VideoQualityTest.

Configure FlexFEC in VideoQualityTest.

Will be used by full stack tests and video_loopback.

BUG=webrtc:5654
Committed: https://crrev.com/1293acae18cb4715b9848d97c9d6794fb7cfd621
Cr-Commit-Position: refs/heads/master@{#15114}

[brandtr, 2016-11-15 11:29:16]

Check pointer instead.

[brandtr, 2016-11-15 11:30:37]

brandtr@webrtc.org changed reviewers:
+ minyue@webrtc.org, stefan@webrtc.org

[brandtr, 2016-11-15 11:30:38]

Hey stefan and minyue,

Please have a look at this change, which adds support for FlexFEC in
VideoQualityTest. At the same time, I'm moving some configuration from
RunWithRenderers to SetupVideo, as per minyue's TODO.

[stefan-webrtc, 2016-11-15 11:52:32]

lgtm % nit

https://codereview.webrtc.org/2500373002/diff/20001/webrtc/video/video_qualit...
File webrtc/video/video_quality_test.cc (right):

https://codereview.webrtc.org/2500373002/diff/20001/webrtc/video/video_qualit...
webrtc/video/video_quality_test.cc:1073: flexfec_receive_config.flexfec_ssrc =
kFlexfecSendSsrc;
Set this to video_send_config_.rtp.flexfec.flexfec_ssrc instead. Same for
payload type.

[minyue-webrtc, 2016-11-15 12:16:58]

lgtm % nit

https://codereview.webrtc.org/2500373002/diff/20001/webrtc/video/full_stack.cc
File webrtc/video/full_stack.cc (right):

https://codereview.webrtc.org/2500373002/diff/20001/webrtc/video/full_stack.c...
webrtc/video/full_stack.cc:28: foreman_cif.video = {true,  352,          288,
30, 700000, 700000, 700000,
I think better not to clang format this file.

[brandtr, 2016-11-15 13:14:40]

https://codereview.webrtc.org/2500373002/diff/20001/webrtc/video/full_stack.cc
File webrtc/video/full_stack.cc (right):

https://codereview.webrtc.org/2500373002/diff/20001/webrtc/video/full_stack.c...
webrtc/video/full_stack.cc:28: foreman_cif.video = {true,  352,          288,
30, 700000, 700000, 700000,
On 2016/11/15 12:16:58, minyue-webrtc wrote:
> I think better not to clang format this file.

Reverted :)

[brandtr, 2016-11-15 15:18:56]

The CQ bit was checked by brandtr@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-15 15:19:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-11-15 15:39:46]

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

[commit-bot: I haz the power, 2016-11-15 15:39:47]

Dry run: This issue passed the CQ dry run.

[brandtr, 2016-11-16 08:55:49]

Patchset #5 (id:80001) has been deleted

[brandtr, 2016-11-16 09:01:19]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2016-11-16 09:01:20]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org,
minyue@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2500373002/#ps100001 (title:
"Actually create FlexfecReceiveStream in RunWithRenderers.")

[commit-bot: I haz the power, 2016-11-16 09:01:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-11-16 09:11:43]

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

[commit-bot: I haz the power, 2016-11-16 09:11:44]

Try jobs failed on following builders:
  mac_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_baremetal/builds/16246)

[brandtr, 2016-11-17 06:37:05]

The CQ bit was checked by brandtr@webrtc.org

[commit-bot: I haz the power, 2016-11-17 06:37:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-11-17 06:47:34]

Committed patchset #5 (id:100001)

[commit-bot: I haz the power, 2016-11-17 06:47:42]

Description was changed from

==========
Configure FlexFEC in VideoQualityTest.

Will be used by full stack tests and video_loopback.

BUG=webrtc:5654
==========

to

==========
Configure FlexFEC in VideoQualityTest.

Will be used by full stack tests and video_loopback.

BUG=webrtc:5654

Committed: https://crrev.com/1293acae18cb4715b9848d97c9d6794fb7cfd621
Cr-Commit-Position: refs/heads/master@{#15114}
==========

[commit-bot: I haz the power, 2016-11-17 06:47:43]

Patchset 5 (id:??) landed as
https://crrev.com/1293acae18cb4715b9848d97c9d6794fb7cfd621
Cr-Commit-Position: refs/heads/master@{#15114}