Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(758)

Unified Diff: webrtc/call/rampup_tests.cc

Issue 2749803002: Enable the bayesian bitrate estimator by default. (Closed)
Patch Set: AppRTC fixes. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | webrtc/examples/objc/AppRTCMobile/ios/ARDAppDelegate.m » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/call/rampup_tests.cc
diff --git a/webrtc/call/rampup_tests.cc b/webrtc/call/rampup_tests.cc
index 292edc60836b4533d7ce5cb2c45fbee869821bd6..0eb6b0fd8717264097c44c979ff743f764d407f9 100644
--- a/webrtc/call/rampup_tests.cc
+++ b/webrtc/call/rampup_tests.cc
@@ -25,6 +25,7 @@ static const int kExpectedHighVideoBitrateBps = 80000;
static const int kExpectedHighAudioBitrateBps = 30000;
static const int kLowBandwidthLimitBps = 20000;
static const int kExpectedLowBitrateBps = 20000;
+static const int kUnlimitedCapacityBps = 10000000;
std::vector<uint32_t> GenerateSsrcs(size_t num_streams, uint32_t ssrc_offset) {
std::vector<uint32_t> ssrcs;
@@ -396,8 +397,9 @@ RampUpDownUpTester::RampUpDownUpTester(size_t num_video_streams,
rtx,
red,
report_perf_stats),
- link_rates_({GetHighLinkCapacity(), kLowBandwidthLimitBps / 1000,
- GetHighLinkCapacity(), 0}),
+ link_rates_({GetExpectedHighBitrate() / 1000,
+ kLowBandwidthLimitBps / 1000, kUnlimitedCapacityBps / 1000,
+ 0}),
test_state_(kFirstRampup),
next_state_(kTransitionToNextState),
state_start_ms_(clock_->TimeInMilliseconds()),
@@ -467,10 +469,6 @@ int RampUpDownUpTester::GetExpectedHighBitrate() const {
return expected_bitrate_bps;
}
-int RampUpDownUpTester::GetHighLinkCapacity() const {
- return 4 * GetExpectedHighBitrate() / (3 * 1000);
-}
-
size_t RampUpDownUpTester::GetFecBytes() const {
size_t flex_fec_bytes = 0;
if (num_flexfec_streams_ > 0) {
« no previous file with comments | « no previous file | webrtc/examples/objc/AppRTCMobile/ios/ARDAppDelegate.m » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698