DescriptionMake |rtcp_send_transport| mandatory in FlexfecReceiveStream::Config.
That object will be used when we enable RTCP reporting from FlexfecReceiveStream.
Other related changes:
- Stop using FlexfecConfig (from config.h) at receive side in WebRtcVideoEngine2.
- Add a IsCompleteAndEnabled() method to FlexfecReceiveStream::Config, to be
used in WebRtcVideoEngine2.
- Centralize the construction of the FlexfecReceiveStream::Config in unit tests.
This will make future additions to the unit tests cleaner.
- Simplify setup for receiving FlexFEC in VideoQualityTest.
BUG=webrtc:5654
Review-Url: https://codereview.webrtc.org/2589713003
Cr-Commit-Position: refs/heads/master@{#16059}
Committed: https://chromium.googlesource.com/external/webrtc/+/8313a6fa8f78d18e0f5e2bff31086da01d0db4bf
Patch Set 1 #Patch Set 2 : Rebase. #
Total comments: 4
Patch Set 3 : Rebase. #Patch Set 4 : philipel comments 1. #Patch Set 5 : Rebase onto master after revert of https://codereview.webrtc.org/2553863003/ #Patch Set 6 : Mini fixes to unit test. #Patch Set 7 : Rebase. #Patch Set 8 : Matching FlexfecReceiveStream configs are only created by RunBaseTest, not by CallTest... #Patch Set 9 : Rebase. #Patch Set 10 : Clarify config setup in VideoQualityTest a bit. #
Total comments: 4
Patch Set 11 : Rebase. #Patch Set 12 : holmer comments 1. #Patch Set 13 : Rebase. #
Total comments: 1
Patch Set 14 : Rebase. #
Dependent Patchsets: Messages
Total messages: 41 (15 generated)
|