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

Side by Side Diff: webrtc/call/bitrate_estimator_tests.cc

Issue 1604563002: Add send-side BWE to WebRtcVoiceEngine under a finch experiment. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Remove RecreateAudioReceiveStream(). Created 4 years, 10 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 #include <functional> 10 #include <functional>
(...skipping 172 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 183
184 if (receive_audio) { 184 if (receive_audio) {
185 AudioReceiveStream::Config receive_config; 185 AudioReceiveStream::Config receive_config;
186 receive_config.rtp.remote_ssrc = test_->video_send_config_.rtp.ssrcs[0]; 186 receive_config.rtp.remote_ssrc = test_->video_send_config_.rtp.ssrcs[0];
187 // Bogus non-default id to prevent hitting a RTC_DCHECK when creating 187 // Bogus non-default id to prevent hitting a RTC_DCHECK when creating
188 // the AudioReceiveStream. Every receive stream has to correspond to 188 // the AudioReceiveStream. Every receive stream has to correspond to
189 // an underlying channel id. 189 // an underlying channel id.
190 receive_config.voe_channel_id = 0; 190 receive_config.voe_channel_id = 0;
191 receive_config.rtp.extensions.push_back( 191 receive_config.rtp.extensions.push_back(
192 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId)); 192 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId));
193 receive_config.combined_audio_video_bwe = true;
194 audio_receive_stream_ = 193 audio_receive_stream_ =
195 test_->receiver_call_->CreateAudioReceiveStream(receive_config); 194 test_->receiver_call_->CreateAudioReceiveStream(receive_config);
196 } else { 195 } else {
197 VideoReceiveStream::Decoder decoder; 196 VideoReceiveStream::Decoder decoder;
198 decoder.decoder = &fake_decoder_; 197 decoder.decoder = &fake_decoder_;
199 decoder.payload_type = 198 decoder.payload_type =
200 test_->video_send_config_.encoder_settings.payload_type; 199 test_->video_send_config_.encoder_settings.payload_type;
201 decoder.payload_name = 200 decoder.payload_name =
202 test_->video_send_config_.encoder_settings.payload_name; 201 test_->video_send_config_.encoder_settings.payload_name;
203 test_->receive_config_.decoders.clear(); 202 test_->receive_config_.decoders.clear();
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
266 265
267 TEST_F(BitrateEstimatorTest, InstantiatesTOFPerDefaultForVideo) { 266 TEST_F(BitrateEstimatorTest, InstantiatesTOFPerDefaultForVideo) {
268 video_send_config_.rtp.extensions.push_back( 267 video_send_config_.rtp.extensions.push_back(
269 RtpExtension(RtpExtension::kTOffset, kTOFExtensionId)); 268 RtpExtension(RtpExtension::kTOffset, kTOFExtensionId));
270 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 269 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
271 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 270 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
272 streams_.push_back(new Stream(this, false)); 271 streams_.push_back(new Stream(this, false));
273 EXPECT_TRUE(receiver_log_.Wait()); 272 EXPECT_TRUE(receiver_log_.Wait());
274 } 273 }
275 274
276 TEST_F(BitrateEstimatorTest, ImmediatelySwitchToASTForAudio) {
the sun 2016/01/26 10:26:55 Do you have more cleanup to do if AST is now depre
stefan-webrtc 2016/01/26 11:40:54 There is a lot of clean up that can be done, but o
277 video_send_config_.rtp.extensions.push_back(
278 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId));
279 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
280 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
281 receiver_log_.PushExpectedLogLine("Switching to absolute send time RBE.");
282 receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);
283 streams_.push_back(new Stream(this, true));
284 EXPECT_TRUE(receiver_log_.Wait());
285 }
286
287 TEST_F(BitrateEstimatorTest, ImmediatelySwitchToASTForVideo) { 275 TEST_F(BitrateEstimatorTest, ImmediatelySwitchToASTForVideo) {
288 video_send_config_.rtp.extensions.push_back( 276 video_send_config_.rtp.extensions.push_back(
289 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId)); 277 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId));
290 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 278 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
291 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 279 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
292 receiver_log_.PushExpectedLogLine("Switching to absolute send time RBE."); 280 receiver_log_.PushExpectedLogLine("Switching to absolute send time RBE.");
293 receiver_log_.PushExpectedLogLine(kAbsSendTimeLog); 281 receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);
294 streams_.push_back(new Stream(this, false)); 282 streams_.push_back(new Stream(this, false));
295 EXPECT_TRUE(receiver_log_.Wait()); 283 EXPECT_TRUE(receiver_log_.Wait());
296 } 284 }
297 285
298 TEST_F(BitrateEstimatorTest, SwitchesToASTForAudio) {
299 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
300 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
301 streams_.push_back(new Stream(this, true));
302 EXPECT_TRUE(receiver_log_.Wait());
303
304 video_send_config_.rtp.extensions.push_back(
305 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId));
306 receiver_log_.PushExpectedLogLine("Switching to absolute send time RBE.");
307 receiver_log_.PushExpectedLogLine(kAbsSendTimeLog);
308 streams_.push_back(new Stream(this, true));
309 EXPECT_TRUE(receiver_log_.Wait());
310 }
311
312 TEST_F(BitrateEstimatorTest, SwitchesToASTForVideo) { 286 TEST_F(BitrateEstimatorTest, SwitchesToASTForVideo) {
313 video_send_config_.rtp.extensions.push_back( 287 video_send_config_.rtp.extensions.push_back(
314 RtpExtension(RtpExtension::kTOffset, kTOFExtensionId)); 288 RtpExtension(RtpExtension::kTOffset, kTOFExtensionId));
315 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 289 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
316 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 290 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
317 streams_.push_back(new Stream(this, false)); 291 streams_.push_back(new Stream(this, false));
318 EXPECT_TRUE(receiver_log_.Wait()); 292 EXPECT_TRUE(receiver_log_.Wait());
319 293
320 video_send_config_.rtp.extensions[0] = 294 video_send_config_.rtp.extensions[0] =
321 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId); 295 RtpExtension(RtpExtension::kAbsSendTime, kASTExtensionId);
(...skipping 22 matching lines...) Expand all
344 RtpExtension(RtpExtension::kTOffset, kTOFExtensionId); 318 RtpExtension(RtpExtension::kTOffset, kTOFExtensionId);
345 receiver_log_.PushExpectedLogLine( 319 receiver_log_.PushExpectedLogLine(
346 "WrappingBitrateEstimator: Switching to transmission time offset RBE."); 320 "WrappingBitrateEstimator: Switching to transmission time offset RBE.");
347 receiver_log_.PushExpectedLogLine(kSingleStreamLog); 321 receiver_log_.PushExpectedLogLine(kSingleStreamLog);
348 streams_.push_back(new Stream(this, false)); 322 streams_.push_back(new Stream(this, false));
349 streams_[0]->StopSending(); 323 streams_[0]->StopSending();
350 streams_[1]->StopSending(); 324 streams_[1]->StopSending();
351 EXPECT_TRUE(receiver_log_.Wait()); 325 EXPECT_TRUE(receiver_log_.Wait());
352 } 326 }
353 } // namespace webrtc 327 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698