admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make BitrateAllocator::transmission_max_bitrate_multiplier_ const.

Browse Source 
To make this work, also make the private method
GetTransmissionMaxBitrateMultiplier static.

Bug: None
Change-Id: I1777602b89d404ecbc9956acf9a9c7f852d69911
Reviewed-on: https://webrtc-review.googlesource.com/72440
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23023}
This commit is contained in:
Niels Möller 2018-04-25 14:03:46 +02:00 committed by Commit Bot
parent 5bb98971ce
commit 74e5f804e4
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
call/bitrate_allocator.cc
View File

@ -72,6 +72,7 @@ BitrateAllocator::~BitrateAllocator() {
num_pause_events_);
}
// static
uint8_t BitrateAllocator::GetTransmissionMaxBitrateMultiplier() {
uint64_t multiplier = strtoul(webrtc::field_trial::FindFullName(
"WebRTC-TransmissionMaxBitrateMultiplier")

4
call/bitrate_allocator.h
View File

@ -219,7 +219,7 @@ class BitrateAllocator : public BitrateAllocatorInterface {
// TODO(bugs.webrtc.org/8541): May be worth to refactor to keep this logic in
// video send stream. Similar logic is implemented in
// AudioPriorityBitrateAllocationStrategy.
uint8_t GetTransmissionMaxBitrateMultiplier();
static uint8_t GetTransmissionMaxBitrateMultiplier();
rtc::SequencedTaskChecker sequenced_checker_;
LimitObserver* const limit_observer_ RTC_GUARDED_BY(&sequenced_checker_);
@ -240,7 +240,7 @@ class BitrateAllocator : public BitrateAllocatorInterface {
bool has_packet_feedback_ RTC_GUARDED_BY(&sequenced_checker_);
std::unique_ptr<rtc::BitrateAllocationStrategy> bitrate_allocation_strategy_
RTC_GUARDED_BY(&sequenced_checker_);
uint8_t transmission_max_bitrate_multiplier_;
const uint8_t transmission_max_bitrate_multiplier_;
};
} // namespace webrtc