mollyim/webrtc
1
0
Fork 0
mirror of https://github.com/mollyim/webrtc.git synced 2025-05-12 21:30:45 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

In RtpBitrateConfigurator ignore new parameters when set to default values.

Browse source 
Bug: webrtc:11263
Change-Id: Ia7539c7c142b059d0295849b916439bb647f112d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/162207
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#30191}
This commit is contained in:
Danil Chapovalov 2019-12-16 15:59:02 +01:00 committed by Commit Bot
parent 26e1b7ac01
commit bcbdeedd43
2 changed files with 23 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
call/rtp_bitrate_configurator.cc
View file

@ -67,8 +67,12 @@ RtpBitrateConfigurator::UpdateWithSdpParameters(
bitrate_config.start_bitrate_bps !=
base_bitrate_config_.start_bitrate_bps) {
new_start.emplace(bitrate_config.start_bitrate_bps);
base_bitrate_config_.start_bitrate_bps = bitrate_config.start_bitrate_bps;
}
base_bitrate_config_ = bitrate_config;
if (bitrate_config.min_bitrate_bps > 0)
base_bitrate_config_.min_bitrate_bps = bitrate_config.min_bitrate_bps;
if (bitrate_config.max_bitrate_bps > 0)
base_bitrate_config_.max_bitrate_bps = bitrate_config.max_bitrate_bps;
return UpdateConstraints(new_start);
}

19
call/rtp_bitrate_configurator_unittest.cc
View file

@ -27,7 +27,7 @@ class RtpBitrateConfiguratorTest : public ::testing::Test {
absl::optional<int> max_bitrate_bps) {
absl::optional<BitrateConstraints> result =
configurator_->UpdateWithSdpParameters(bitrate_config);
EXPECT_TRUE(result.has_value());
ASSERT_TRUE(result.has_value());
if (start_bitrate_bps.has_value())
EXPECT_EQ(result->start_bitrate_bps, start_bitrate_bps);
if (min_bitrate_bps.has_value())
@ -231,6 +231,23 @@ TEST_F(RtpBitrateConfiguratorTest, NewConfigWithNoChangesDoesNotCallNewConfig) {
EXPECT_FALSE(configurator_->UpdateWithSdpParameters(config2).has_value());
}
TEST_F(RtpBitrateConfiguratorTest,
NewConfigWithUnsetMinAndMaxDoesNotCallNewConfig) {
BitrateConstraints config1;
config1.min_bitrate_bps = 100'000;
config1.start_bitrate_bps = 1'000;
config1.max_bitrate_bps = 1'000'000;
BitrateConstraints config2;
config2.min_bitrate_bps = 0;
config2.start_bitrate_bps = -1;
config2.max_bitrate_bps = -1;
configurator_->UpdateWithSdpParameters(config1);
// The second call should return nothing because it doesn't change any values.
EXPECT_EQ(configurator_->UpdateWithSdpParameters(config2), absl::nullopt);
}
// If config changes the max, but not the effective max,
// new config shouldn't be returned, to avoid unnecessary encoder
// reconfigurations.