DescriptionSplit CongestionController into send- and receive-side classes.
New class ReceiveSideCongestionController, extracted from CongestionController, and responsible for the
OnReceivedPacket processing.
Rest of the CongestionController moved to a new class
SendSideCongestionController.
To avoid breaking applications, CongestionController is redefined
as a union of these two classes, with no intended change in behavior.
With one exception: CongestionController::SetBweBitrates used to call
remote_bitrate_estimator_.SetMinBitrate, but after remote_bitrate_estimator_ was moved to ReceiveSideCongestionController,
it no longer does this.
BUG=webrtc:6847
Review-Url: https://codereview.webrtc.org/2752233002
Cr-Commit-Position: refs/heads/master@{#17321}
Committed: https://chromium.googlesource.com/external/webrtc/+/559af38a15f6590926759440002512b70d01ba58
Patch Set 1 #Patch Set 2 : New class SendSideCongestionController, define CongestionController as a union. #Patch Set 3 : Reformatted, and fixed a few presubmit warnings. #
Total comments: 4
Patch Set 4 : Improve comments. Move declaration of destructor. #
Total comments: 4
Patch Set 5 : Deleted ReceiveSideCongestionController::OnNetworkRouteChanged. #
Total comments: 4
Patch Set 6 : Rebased. #Patch Set 7 : Added method ReceiveSideCongestionController::OnBitrateChanged. #
Total comments: 2
Patch Set 8 : Use variable names receive_side_cc and send_side_cc. #Messages
Total messages: 31 (12 generated)
|