Revert of Use RtxReceiveStream.

Revert of Use RtxReceiveStream. (patchset #5 id:320001 of https://codereview.webrtc.org/3006063002/ )

Reason for revert:
This change appears to break ulpfec, with severe regressions, e.g., for webrtc_perf_test FullStackTest.ForemanCifPlr5Ulpfec

Original issue's description:
> Reland of Use RtxReceiveStream. (patchset #1 id:1 of https://codereview.webrtc.org/3010983002/ )
> 
> Reason for revert:
> Intend to fix perf failures and reland.
> 
> Original issue's description:
> > Revert of Use RtxReceiveStream. (patchset #5 id:80001 of https://codereview.webrtc.org/3008773002/ )
> > 
> > Reason for revert:
> > A few perf tests broken, including
> > 
> > RampUpTest.UpDownUpAbsSendTimeSimulcastRedRtx
> > RampUpTest.UpDownUpTransportSequenceNumberRtx
> > RampUpTest.UpDownUpTransportSequenceNumberPacketLoss
> > 
> > 
> > Original issue's description:
> > > Use RtxReceiveStream.
> > > 
> > > This also has the beneficial side-effect that when a media stream
> > > which is protected by FlexFEC receives an RTX retransmission, the
> > > retransmitted media packet is passed into the FlexFEC machinery,
> > > which should improve its ability to recover packets via FEC.
> > >  
> > > BUG=webrtc:7135
> > > 
> > > Review-Url: https://codereview.webrtc.org/3008773002
> > > Cr-Commit-Position: refs/heads/master@{#19649}
> > > Committed: https://chromium.googlesource.com/external/webrtc/+/5c0f6c62ea3b1d2c43f8fc152961af27033475f7
> > 
> > TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:7135
> > 
> > Review-Url: https://codereview.webrtc.org/3010983002
> > Cr-Commit-Position: refs/heads/master@{#19653}
> > Committed: https://chromium.googlesource.com/external/webrtc/+/3c39c0137afa274d1d524b150b50304b38a2847b
> 
> TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> BUG=webrtc:7135
> 
> Review-Url: https://codereview.webrtc.org/3006063002
> Cr-Commit-Position: refs/heads/master@{#19715}
> Committed: https://chromium.googlesource.com/external/webrtc/+/35713eaf565c0fef07c8afc158d7b8fdf7ec3d78

TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7135
Review-Url: https://codereview.webrtc.org/3007303002
Cr-Commit-Position: refs/heads/master@{#19744}
Committed: https://chromium.googlesource.com/external/webrtc/+/8e7eee035178a7f10e19883681b5eaa4a7523107

[nisse-webrtc, 2017-09-08 12:24:36]

The CQ bit was checked by nisse@webrtc.org

[nisse-webrtc, 2017-09-08 12:24:37]

Created Revert of Use RtxReceiveStream.

[commit-bot: I haz the power, 2017-09-08 12:24:50]

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-09-08 12:51:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1504873476787170, "parent_rev":
"66d07c54fef7b0ddd5418f05096757e2acc8308e", "commit_rev":
"8e7eee035178a7f10e19883681b5eaa4a7523107"}

[commit-bot: I haz the power, 2017-09-08 12:51:59]

Description was changed from

==========
Revert of Use RtxReceiveStream. (patchset #5 id:320001 of
https://codereview.webrtc.org/3006063002/ )

Reason for revert:
This change appears to break ulpfec, with severe regressions, e.g., for
webrtc_perf_test FullStackTest.ForemanCifPlr5Ulpfec

Original issue's description:
> Reland of Use RtxReceiveStream. (patchset #1 id:1 of
https://codereview.webrtc.org/3010983002/ )
> 
> Reason for revert:
> Intend to fix perf failures and reland.
> 
> Original issue's description:
> > Revert of Use RtxReceiveStream. (patchset #5 id:80001 of
https://codereview.webrtc.org/3008773002/ )
> > 
> > Reason for revert:
> > A few perf tests broken, including
> > 
> > RampUpTest.UpDownUpAbsSendTimeSimulcastRedRtx
> > RampUpTest.UpDownUpTransportSequenceNumberRtx
> > RampUpTest.UpDownUpTransportSequenceNumberPacketLoss
> > 
> > 
> > Original issue's description:
> > > Use RtxReceiveStream.
> > > 
> > > This also has the beneficial side-effect that when a media stream
> > > which is protected by FlexFEC receives an RTX retransmission, the
> > > retransmitted media packet is passed into the FlexFEC machinery,
> > > which should improve its ability to recover packets via FEC.
> > >  
> > > BUG=webrtc:7135
> > > 
> > > Review-Url: https://codereview.webrtc.org/3008773002
> > > Cr-Commit-Position: refs/heads/master@{#19649}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/5c0f6c62ea3b1d2c43f8fc152...
> > 
> >
TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:7135
> > 
> > Review-Url: https://codereview.webrtc.org/3010983002
> > Cr-Commit-Position: refs/heads/master@{#19653}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/3c39c0137afa274d1d524b150...
> 
>
TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> BUG=webrtc:7135
> 
> Review-Url: https://codereview.webrtc.org/3006063002
> Cr-Commit-Position: refs/heads/master@{#19715}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/35713eaf565c0fef07c8afc15...

TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7135
==========

to

==========
Revert of Use RtxReceiveStream. (patchset #5 id:320001 of
https://codereview.webrtc.org/3006063002/ )

Reason for revert:
This change appears to break ulpfec, with severe regressions, e.g., for
webrtc_perf_test FullStackTest.ForemanCifPlr5Ulpfec

Original issue's description:
> Reland of Use RtxReceiveStream. (patchset #1 id:1 of
https://codereview.webrtc.org/3010983002/ )
> 
> Reason for revert:
> Intend to fix perf failures and reland.
> 
> Original issue's description:
> > Revert of Use RtxReceiveStream. (patchset #5 id:80001 of
https://codereview.webrtc.org/3008773002/ )
> > 
> > Reason for revert:
> > A few perf tests broken, including
> > 
> > RampUpTest.UpDownUpAbsSendTimeSimulcastRedRtx
> > RampUpTest.UpDownUpTransportSequenceNumberRtx
> > RampUpTest.UpDownUpTransportSequenceNumberPacketLoss
> > 
> > 
> > Original issue's description:
> > > Use RtxReceiveStream.
> > > 
> > > This also has the beneficial side-effect that when a media stream
> > > which is protected by FlexFEC receives an RTX retransmission, the
> > > retransmitted media packet is passed into the FlexFEC machinery,
> > > which should improve its ability to recover packets via FEC.
> > >  
> > > BUG=webrtc:7135
> > > 
> > > Review-Url: https://codereview.webrtc.org/3008773002
> > > Cr-Commit-Position: refs/heads/master@{#19649}
> > > Committed:
https://chromium.googlesource.com/external/webrtc/+/5c0f6c62ea3b1d2c43f8fc152...
> > 
> >
TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
> > # Skipping CQ checks because original CL landed less than 1 days ago.
> > NOPRESUBMIT=true
> > NOTREECHECKS=true
> > NOTRY=true
> > BUG=webrtc:7135
> > 
> > Review-Url: https://codereview.webrtc.org/3010983002
> > Cr-Commit-Position: refs/heads/master@{#19653}
> > Committed:
https://chromium.googlesource.com/external/webrtc/+/3c39c0137afa274d1d524b150...
> 
>
TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> BUG=webrtc:7135
> 
> Review-Url: https://codereview.webrtc.org/3006063002
> Cr-Commit-Position: refs/heads/master@{#19715}
> Committed:
https://chromium.googlesource.com/external/webrtc/+/35713eaf565c0fef07c8afc15...

TBR=brandtr@webrtc.org,danilchap@webrtc.org,stefan@webrtc.org,magjed@webrtc.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=webrtc:7135

Review-Url: https://codereview.webrtc.org/3007303002
Cr-Commit-Position: refs/heads/master@{#19744}
Committed:
https://chromium.googlesource.com/external/webrtc/+/8e7eee035178a7f10e1988368...
==========

[commit-bot: I haz the power, 2017-09-08 12:52:02]

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

[nisse-webrtc, 2017-09-11 07:06:49]

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

The reason for reverting is: Identified a configuration problem in the video
quality tests. Intend to fix and reland..