Revert of Make RTX pt/apt reconfigurable by calling WebRtcVideoChannel2::SetRecvParameters.

Revert of Make RTX pt/apt reconfigurable by calling WebRtcVideoChannel2::SetRecvParameters. (patchset #7 id:160001 of https://codereview.webrtc.org/2646073004/ )

Reason for revert:
Breaks internal downstream project.

Original issue's description:
> Make RTX pt/apt reconfigurable by calling WebRtcVideoChannel2::SetRecvParameters.
> 
> Prior to this CL, received RTX (associated) payload types were only configured
> when WebRtcVideoChannel2::AddRecvStream was called. In the same method, the RTX
> SSRC was set up.
> 
> After this CL, the RTX (associated) payload types are set in
> WebRtcVideoChannel2::SetRecvParameters, which is the appropriate place to set
> them. The RTX SSRC is still set in WebRtcVideoChannel2::AddRecvStream, since
> that is the code path that sets other SSRCs.
> 
> As part of this fix, the VideoReceiveStream::Config::Rtp struct is changed.
> We remove the possibility for each video payload type to have an associated
> specific RTX SSRC. Although the config previously allowed for this, all payload
> types always had the same RTX SSRC set, and the underlying RtpPayloadRegistry
> did not support multiple SSRCs. This change to the config struct should thus not
> have any functional impact. The change does however affect the RtcEventLog, since
> that is used for storing the VideoReceiveStream::Configs. For simplicity,
> this CL does not change the event log proto definitions, instead duplicating
> the serialized RTX SSRCs such that they fit in the existing proto definition.
> 
> BUG=webrtc:7011
> 
> Review-Url: https://codereview.webrtc.org/2646073004
> Cr-Commit-Position: refs/heads/master@{#16302}
> Committed: https://chromium.googlesource.com/external/webrtc/+/fe2bef39cd2a5c891a49f7320514fb04324dc66c

TBR=stefan@webrtc.org,magjed@webrtc.org,terelius@webrtc.org,brandtr@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7011
Review-Url: https://codereview.webrtc.org/2649323010
Cr-Commit-Position: refs/heads/master@{#16307}
Committed: https://chromium.googlesource.com/external/webrtc/+/e4974953ce0d03a60fae7659b199a6a62a79fa30

[kjellander (google.com), 2017-01-26 21:20:55]

The CQ bit was checked by kjellander@google.com

[kjellander (google.com), 2017-01-26 21:20:56]

Created Revert of Make RTX pt/apt reconfigurable by calling
WebRtcVideoChannel2::SetRecvParameters.

[commit-bot: I haz the power, 2017-01-26 21:21:01]

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, 2017-01-26 21:21:02]

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

[commit-bot: I haz the power, 2017-01-26 21:21:03]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-webrtc-committers".
Note that this has nothing to do with OWNERS files.

[kjellander (google.com), 2017-01-26 21:21:06]

The CQ bit was unchecked by kjellander@google.com

[kjellander_webrtc, 2017-01-26 21:22:22]

kjellander@webrtc.org changed reviewers:
+ kjellander@webrtc.org

[kjellander_webrtc, 2017-01-26 21:22:23]

ah, wrong account again. lgtm with my webrtc.org account

[kjellander_webrtc, 2017-01-26 21:22:28]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2017-01-26 21:22:32]

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, 2017-01-26 21:22:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485465748729550, "parent_rev":
"71ecaad3ea3fd3d9b0d9fa802b7ee86bd1bae3a1", "commit_rev":
"e4974953ce0d03a60fae7659b199a6a62a79fa30"}

[commit-bot: I haz the power, 2017-01-26 21:22:41]

Description was changed from

==========
Revert of Make RTX pt/apt reconfigurable by calling
WebRtcVideoChannel2::SetRecvParameters. (patchset #7 id:160001 of
https://codereview.webrtc.org/2646073004/ )

Reason for revert:
Breaks internal downstream project.

Original issue's description:
> Make RTX pt/apt reconfigurable by calling
WebRtcVideoChannel2::SetRecvParameters.
> 
> Prior to this CL, received RTX (associated) payload types were only configured
> when WebRtcVideoChannel2::AddRecvStream was called. In the same method, the
RTX
> SSRC was set up.
> 
> After this CL, the RTX (associated) payload types are set in
> WebRtcVideoChannel2::SetRecvParameters, which is the appropriate place to set
> them. The RTX SSRC is still set in WebRtcVideoChannel2::AddRecvStream, since
> that is the code path that sets other SSRCs.
> 
> As part of this fix, the VideoReceiveStream::Config::Rtp struct is changed.
> We remove the possibility for each video payload type to have an associated
> specific RTX SSRC. Although the config previously allowed for this, all
payload
> types always had the same RTX SSRC set, and the underlying RtpPayloadRegistry
> did not support multiple SSRCs. This change to the config struct should thus
not
> have any functional impact. The change does however affect the RtcEventLog,
since
> that is used for storing the VideoReceiveStream::Configs. For simplicity,
> this CL does not change the event log proto definitions, instead duplicating
> the serialized RTX SSRCs such that they fit in the existing proto definition.
> 
> BUG=webrtc:7011
> 
> Review-Url: https://codereview.webrtc.org/2646073004
> Cr-Commit-Position: refs/heads/master@{#16302}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fe2bef39cd2a5c891a49f7320...

TBR=stefan@webrtc.org,magjed@webrtc.org,terelius@webrtc.org,brandtr@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7011
==========

to

==========
Revert of Make RTX pt/apt reconfigurable by calling
WebRtcVideoChannel2::SetRecvParameters. (patchset #7 id:160001 of
https://codereview.webrtc.org/2646073004/ )

Reason for revert:
Breaks internal downstream project.

Original issue's description:
> Make RTX pt/apt reconfigurable by calling
WebRtcVideoChannel2::SetRecvParameters.
> 
> Prior to this CL, received RTX (associated) payload types were only configured
> when WebRtcVideoChannel2::AddRecvStream was called. In the same method, the
RTX
> SSRC was set up.
> 
> After this CL, the RTX (associated) payload types are set in
> WebRtcVideoChannel2::SetRecvParameters, which is the appropriate place to set
> them. The RTX SSRC is still set in WebRtcVideoChannel2::AddRecvStream, since
> that is the code path that sets other SSRCs.
> 
> As part of this fix, the VideoReceiveStream::Config::Rtp struct is changed.
> We remove the possibility for each video payload type to have an associated
> specific RTX SSRC. Although the config previously allowed for this, all
payload
> types always had the same RTX SSRC set, and the underlying RtpPayloadRegistry
> did not support multiple SSRCs. This change to the config struct should thus
not
> have any functional impact. The change does however affect the RtcEventLog,
since
> that is used for storing the VideoReceiveStream::Configs. For simplicity,
> this CL does not change the event log proto definitions, instead duplicating
> the serialized RTX SSRCs such that they fit in the existing proto definition.
> 
> BUG=webrtc:7011
> 
> Review-Url: https://codereview.webrtc.org/2646073004
> Cr-Commit-Position: refs/heads/master@{#16302}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/fe2bef39cd2a5c891a49f7320...

TBR=stefan@webrtc.org,magjed@webrtc.org,terelius@webrtc.org,brandtr@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:7011

Review-Url: https://codereview.webrtc.org/2649323010
Cr-Commit-Position: refs/heads/master@{#16307}
Committed:
https://chromium.googlesource.com/external/webrtc/+/e4974953ce0d03a60fae7659b...
==========

[commit-bot: I haz the power, 2017-01-26 21:22:42]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/e4974953ce0d03a60fae7659b...

[brandtr, 2017-01-27 11:16:03]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2654163006/ by brandtr@webrtc.org.

The reason for reverting is: Downstream project relied on changed struct.

Transition made possible by https://codereview.webrtc.org/2655243006/..