Let ViEEncoder tell VideoSendStream about reconfigurations.

Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to VideoSendStream through the ViEEncoder.
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the encoder video frame size changes. Instead, the number of sending streams is only decided by the number of sending ssrc.

This cl is a preparation for moving encoder reconfiguration due to input video frame size changes from WebRtcVideoSendStream to ViEEncoder.

BUG=webrtc:5687, webrtc:6371
R=mflodman@webrtc.org

Committed: https://crrev.com/512ecb320668bb1a70dd5bf56e6a5732a58f8743
Cr-Commit-Position: refs/heads/master@{#14371}

[perkj_webrtc, 2016-09-14 13:15:06]

Description was changed from

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

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

to

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder. 
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc. 

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder. 

BUG=webrtc:5687
==========

[perkj_webrtc, 2016-09-14 13:15:06]

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

[perkj_webrtc, 2016-09-14 13:33:17]

please?

[perkj_webrtc, 2016-09-16 10:54:02]

Description was changed from

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder. 
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc. 

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder. 

BUG=webrtc:5687
==========

to

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder. 
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc. 

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder. 

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

[perkj_webrtc, 2016-09-16 10:54:48]

Patchset #4 (id:60001) has been deleted

[perkj_webrtc, 2016-09-20 07:24:16]

ping

[perkj_webrtc, 2016-09-23 06:50:12]

Guys, there is more than a week ago since I sent out this for review. Please
review or let me know if there is anyone else who can review it.

[mflodman, 2016-09-23 08:02:56]

LGTM!

[perkj_webrtc, 2016-09-23 09:07:03]

The CQ bit was checked by perkj@webrtc.org

[commit-bot: I haz the power, 2016-09-23 09:07:12]

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-23 09:10:41]

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

[commit-bot: I haz the power, 2016-09-23 09:10:42]

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

[perkj_webrtc, 2016-09-23 11:16:06]

The CQ bit was checked by perkj@webrtc.org

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

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-23 11:30:02]

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

[commit-bot: I haz the power, 2016-09-23 11:30:03]

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

[perkj_webrtc, 2016-09-23 13:52:24]

Description was changed from

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder. 
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc. 

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder. 

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

to

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder.
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc.

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder.

BUG=webrtc:5687, webrtc:6371
R=mflodman@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/512ecb320668bb1a70dd5bf56...
==========

[perkj_webrtc, 2016-09-23 13:52:25]

Committed patchset #4 (id:80001) manually as
512ecb320668bb1a70dd5bf56e6a5732a58f8743 (presubmit successful).

[commit-bot: I haz the power, 2016-09-23 13:52:25]

Description was changed from

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder.
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc.

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder.

BUG=webrtc:5687, webrtc:6371
R=mflodman@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/512ecb320668bb1a70dd5bf56...
==========

to

==========
Let ViEEncoder tell VideoSendStream about reconfigurations.

This cl change so that all encoder configuration changes are reported to
VideoSendStream through the ViEEncoder.
Also, the PayLoadRouter is changed to never stop sending on a an ssrc due to the
encoder video frame size changes. Instead, the number of sending streams is only
decided by the number of sending ssrc.

This cl is a preparation for moving encoder reconfiguration due to input video
frame size changes from WebRtcVideoSendStream to ViEEncoder.

BUG=webrtc:5687, webrtc:6371
R=mflodman@webrtc.org

Committed: https://crrev.com/512ecb320668bb1a70dd5bf56e6a5732a58f8743
Cr-Commit-Position: refs/heads/master@{#14371}
==========

[perkj_webrtc, 2016-09-23 14:04:16]

A revert of this CL (patchset #4 id:80001) has been created in
https://codereview.webrtc.org/2364913002/ by perkj@webrtc.org.

The reason for reverting is: git cl land broke tests. 
Should have done a clean rebase......

[ehmaldonado_webrtc, 2016-09-23 14:35:05]

On 2016/09/23 14:04:16, perkj_webrtc wrote:
> A revert of this CL (patchset #4 id:80001) has been created in
> https://codereview.webrtc.org/2364913002/ by mailto:perkj@webrtc.org.
> 
> The reason for reverting is: git cl land broke tests. 
> Should have done a clean rebase......

I get "No issue exists with that id (2364913002)" when I try to look at it.