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

Side by Side Diff: webrtc/video/rampup_tests.cc

Issue 1433703002: Remove contention between RTCP packets and encoding. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: added TODO in comment Created 5 years, 1 month 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 10
(...skipping 255 matching lines...) Expand 10 before | Expand all | Expand 10 after
266 ReportResult("ramp-up-rtx-padding-sent", rtx_padding_sent, "bytes"); 266 ReportResult("ramp-up-rtx-padding-sent", rtx_padding_sent, "bytes");
267 if (ramp_up_finished_ms_ >= 0) { 267 if (ramp_up_finished_ms_ >= 0) {
268 ReportResult("ramp-up-time", ramp_up_finished_ms_ - test_start_ms_, 268 ReportResult("ramp-up-time", ramp_up_finished_ms_ - test_start_ms_,
269 "milliseconds"); 269 "milliseconds");
270 } 270 }
271 } 271 }
272 272
273 void RampUpTester::PerformTest() { 273 void RampUpTester::PerformTest() {
274 test_start_ms_ = clock_->TimeInMilliseconds(); 274 test_start_ms_ = clock_->TimeInMilliseconds();
275 poller_thread_->Start(); 275 poller_thread_->Start();
276 if (Wait() != kEventSignaled) { 276 EXPECT_EQ(kEventSignaled, Wait())
277 printf("Timed out while waiting for ramp-up to complete."); 277 << "Timed out while waiting for ramp-up to complete.";
278 return;
279 }
280 TriggerTestDone(); 278 TriggerTestDone();
281 poller_thread_->Stop(); 279 poller_thread_->Stop();
282 } 280 }
283 281
284 RampUpDownUpTester::RampUpDownUpTester(size_t num_streams, 282 RampUpDownUpTester::RampUpDownUpTester(size_t num_streams,
285 unsigned int start_bitrate_bps, 283 unsigned int start_bitrate_bps,
286 const std::string& extension_type, 284 const std::string& extension_type,
287 bool rtx, 285 bool rtx,
288 bool red) 286 bool red)
289 : RampUpTester(num_streams, start_bitrate_bps, extension_type, rtx, red), 287 : RampUpTester(num_streams, start_bitrate_bps, extension_type, rtx, red),
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
500 RampUpTester test(3, 0, RtpExtension::kTransportSequenceNumber, true, true); 498 RampUpTester test(3, 0, RtpExtension::kTransportSequenceNumber, true, true);
501 RunBaseTest(&test, FakeNetworkPipe::Config()); 499 RunBaseTest(&test, FakeNetworkPipe::Config());
502 } 500 }
503 501
504 TEST_F(RampUpTest, TransportSequenceNumberSingleStreamWithHighStartBitrate) { 502 TEST_F(RampUpTest, TransportSequenceNumberSingleStreamWithHighStartBitrate) {
505 RampUpTester test(1, 0.9 * kSingleStreamTargetBps, 503 RampUpTester test(1, 0.9 * kSingleStreamTargetBps,
506 RtpExtension::kTransportSequenceNumber, false, false); 504 RtpExtension::kTransportSequenceNumber, false, false);
507 RunBaseTest(&test, FakeNetworkPipe::Config()); 505 RunBaseTest(&test, FakeNetworkPipe::Config());
508 } 506 }
509 } // namespace webrtc 507 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/video_coding/main/source/video_sender.cc ('k') | webrtc/video/video_send_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698