Releand of Let ViEEncoder handle resolution changes.

Releand of Let ViEEncoder handle resolution changes.

The original landed cl is in patchset 1.
The following patchset fix VideoQualityTest as well as fix the case where max_bitrate is set in the SendParams. A unit test is added for that as well.

Original cl description:
Let ViEEncoder handle resolution changes.

This cl move codec reconfiguration due to video frame size changes from WebRtcVideoSendStream to ViEEncoder.

With this change, many variables in WebRtcVideoSendStream no longer need to be locked.

BUG=webrtc:5687, webrtc:6371, webrtc:5332
Committed: https://crrev.com/fa10b557d9051b421a39093ca83302f0fdf04705
Cr-Commit-Position: refs/heads/master@{#14467}

[perkj_webrtc, 2016-09-30 09:39:17]

Description was changed from

==========
Releand of Let ViEEncoder handle resolution changes. (patchset #17 id:340001 of
https://codereview.webrtc.org/2351633002/ )"

This reverts commit 3b703ede8b84267a4ff216aec9296e5dcdd7b6b7.

BUG=
==========

to

==========
Releand of Let ViEEncoder handle resolution changes. 

The original landed cl is in patchset 1. 
The following patchset fix VideoQualityTest as well as fix the case where
max_bitrate is set in the SendParams. A unit test is added for that as well.

Original cl description:
Let ViEEncoder handle resolution changes.

This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.

With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.

BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

[perkj_webrtc, 2016-09-30 09:39:17]

perkj@webrtc.org changed reviewers:
+ mflodman@webrtc.org

[perkj_webrtc, 2016-09-30 09:40:02]

please?

[mflodman, 2016-09-30 10:24:02]

LG, but seems to still be some issue with the rtc unittest.

[perkj_webrtc, 2016-09-30 11:45:33]

On 2016/09/30 10:24:02, mflodman wrote:
> LG, but seems to still be some issue with the rtc unittest.

Nisse had commited a cl in between... Fixed in latest patchset.

[mflodman, 2016-09-30 13:16:00]

LGTM

[mflodman, 2016-09-30 13:16:11]

The CQ bit was checked by mflodman@webrtc.org

[commit-bot: I haz the power, 2016-09-30 13:16:17]

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-09-30 13:45:18]

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

[commit-bot: I haz the power, 2016-09-30 13:45:19]

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

[perkj_webrtc, 2016-10-03 05:35:32]

The CQ bit was checked by perkj@webrtc.org

[commit-bot: I haz the power, 2016-10-03 05:35:35]

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-10-03 05:37:04]

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

[commit-bot: I haz the power, 2016-10-03 05:37:05]

Try jobs failed on following builders:
  android_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_dbg/builds/17261)
  linux_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_baremetal/builds/...)
  mac_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_baremetal/builds/14869)
  mac_compile_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_compile_dbg/builds/...)
  mac_gyp_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_gyp_dbg/builds/1148)

[perkj_webrtc, 2016-10-03 05:55:56]

The CQ bit was checked by perkj@webrtc.org

[perkj_webrtc, 2016-10-03 05:55:57]

The patchset sent to the CQ was uploaded after l-g-t-m from mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2386573002/#ps80001 (title:
"Rebased")

[commit-bot: I haz the power, 2016-10-03 05:56:05]

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-10-03 06:45:29]

Description was changed from

==========
Releand of Let ViEEncoder handle resolution changes. 

The original landed cl is in patchset 1. 
The following patchset fix VideoQualityTest as well as fix the case where
max_bitrate is set in the SendParams. A unit test is added for that as well.

Original cl description:
Let ViEEncoder handle resolution changes.

This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.

With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.

BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

to

==========
Releand of Let ViEEncoder handle resolution changes. 

The original landed cl is in patchset 1. 
The following patchset fix VideoQualityTest as well as fix the case where
max_bitrate is set in the SendParams. A unit test is added for that as well.

Original cl description:
Let ViEEncoder handle resolution changes.

This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.

With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.

BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

[commit-bot: I haz the power, 2016-10-03 06:45:30]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-10-03 06:45:37]

Description was changed from

==========
Releand of Let ViEEncoder handle resolution changes. 

The original landed cl is in patchset 1. 
The following patchset fix VideoQualityTest as well as fix the case where
max_bitrate is set in the SendParams. A unit test is added for that as well.

Original cl description:
Let ViEEncoder handle resolution changes.

This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.

With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.

BUG=webrtc:5687, webrtc:6371, webrtc:5332
==========

to

==========
Releand of Let ViEEncoder handle resolution changes.

The original landed cl is in patchset 1.
The following patchset fix VideoQualityTest as well as fix the case where
max_bitrate is set in the SendParams. A unit test is added for that as well.

Original cl description:
Let ViEEncoder handle resolution changes.

This cl move codec reconfiguration due to video frame size changes from
WebRtcVideoSendStream to ViEEncoder.

With this change, many variables in WebRtcVideoSendStream no longer need to be
locked.

BUG=webrtc:5687, webrtc:6371, webrtc:5332

Committed: https://crrev.com/fa10b557d9051b421a39093ca83302f0fdf04705
Cr-Commit-Position: refs/heads/master@{#14467}
==========

[commit-bot: I haz the power, 2016-10-03 06:45:39]

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