webrtc/call/bitrate_estimator_tests.cc - Issue 2234363002: Removed old probe cluster logic and logic related to ssrcs from DelayBasedBwe.

Unified Diff: webrtc/call/bitrate_estimator_tests.cc

Issue 2234363002: Removed old probe cluster logic and logic related to ssrcs from DelayBasedBwe. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master

Patch Set: Added comment for kFixedSsrc. Created 4 years, 4 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

Index: webrtc/call/bitrate_estimator_tests.cc

diff --git a/webrtc/call/bitrate_estimator_tests.cc b/webrtc/call/bitrate_estimator_tests.cc

index 88da13c864400298fb82668b635de4334c13e1b7..2d69b813b94dffad7e1b7a16b7d124f6f90da084 100644

--- a/webrtc/call/bitrate_estimator_tests.cc

+++ b/webrtc/call/bitrate_estimator_tests.cc

@@ -269,7 +269,6 @@ TEST_F(BitrateEstimatorTest, InstantiatesTOFPerDefaultForVideo) {

video_send_config_.rtp.extensions.push_back(

RtpExtension(RtpExtension::kTimestampOffsetUri, kTOFExtensionId));

receiver_log_.PushExpectedLogLine(kSingleStreamLog);

- receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);

receiver_log_.PushExpectedLogLine(kSingleStreamLog);

streams_.push_back(new Stream(this, false));

EXPECT_TRUE(receiver_log_.Wait());

@@ -279,9 +278,7 @@ TEST_F(BitrateEstimatorTest, ImmediatelySwitchToASTForVideo) {

video_send_config_.rtp.extensions.push_back(

RtpExtension(RtpExtension::kAbsSendTimeUri, kASTExtensionId));

receiver_log_.PushExpectedLogLine(kSingleStreamLog);

- receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);

receiver_log_.PushExpectedLogLine(kSingleStreamLog);

- receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);

receiver_log_.PushExpectedLogLine("Switching to absolute send time RBE.");

stefan-webrtc 2016/08/11 15:38:59 I'm surprised that this doesn't require an abs sen

philipel 2016/08/12 08:31:02 Line 282 and 284 has now been replaced by the Dela

streams_.push_back(new Stream(this, false));

EXPECT_TRUE(receiver_log_.Wait());

@@ -291,14 +288,12 @@ TEST_F(BitrateEstimatorTest, SwitchesToASTForVideo) {

video_send_config_.rtp.extensions.push_back(

RtpExtension(RtpExtension::kTimestampOffsetUri, kTOFExtensionId));

receiver_log_.PushExpectedLogLine(kSingleStreamLog);

- receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);

receiver_log_.PushExpectedLogLine(kSingleStreamLog);

streams_.push_back(new Stream(this, false));

EXPECT_TRUE(receiver_log_.Wait());

video_send_config_.rtp.extensions[0] =

RtpExtension(RtpExtension::kAbsSendTimeUri, kASTExtensionId);

- receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);

receiver_log_.PushExpectedLogLine("Switching to absolute send time RBE.");

streams_.push_back(new Stream(this, false));

EXPECT_TRUE(receiver_log_.Wait());

« no previous file with comments | « no previous file | webrtc/modules/congestion_controller/delay_based_bwe.h » ('j') | no next file with comments »