Chromium Code Reviews| Index: webrtc/call/congestion_controller.cc |
| diff --git a/webrtc/call/congestion_controller.cc b/webrtc/call/congestion_controller.cc |
| index 08cdb11d2fbaf66a385429a66fe6e197cbef4200..793c70c970724a772f7d9234d8de14a24bcb427f 100644 |
| --- a/webrtc/call/congestion_controller.cc |
| +++ b/webrtc/call/congestion_controller.cc |
| @@ -82,11 +82,6 @@ class WrappingBitrateEstimator : public RemoteBitrateEstimator { |
| return rbe_->LatestEstimate(ssrcs, bitrate_bps); |
| } |
| - bool GetStats(ReceiveBandwidthEstimatorStats* output) const override { |
| - CriticalSectionScoped cs(crit_sect_.get()); |
| - return rbe_->GetStats(output); |
| - } |
| - |
| void SetMinBitrate(int min_bitrate_bps) { |
| CriticalSectionScoped cs(crit_sect_.get()); |
| rbe_->SetMinBitrate(min_bitrate_bps); |
| @@ -193,6 +188,12 @@ CongestionController::~CongestionController() { |
| void CongestionController::SetBweBitrates(int min_bitrate_bps, |
| int start_bitrate_bps, |
| int max_bitrate_bps) { |
| + // TODO(holmer): We should make sure the default bitrates are set to 10 kbps, |
| + // and that we don't try to set the min bitrate to 0 from any applications. |
| + // The congestion controller should allow a min bitrate of 0. |
|
pbos-webrtc
2016/02/16 13:44:34
Should this clamp to start and max as well? All mi
stefan-webrtc
2016/02/16 14:57:16
Done.
|
| + const int kMinBitrateBps = 10000; |
| + if (min_bitrate_bps < kMinBitrateBps) |
| + min_bitrate_bps = kMinBitrateBps; |
| if (start_bitrate_bps > 0) |
| bitrate_controller_->SetStartBitrate(start_bitrate_bps); |
| bitrate_controller_->SetMinMaxBitrate(min_bitrate_bps, max_bitrate_bps); |