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

Side by Side Diff: webrtc/modules/congestion_controller/transport_feedback_adapter_unittest.cc

Issue 2693213005: Rename some variables and methods in RTC event log. (Closed)
Patch Set: Unset-upstream Created 3 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) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 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 18 matching lines...) Expand all
29 namespace test { 29 namespace test {
30 30
31 class TransportFeedbackAdapterTest : public ::testing::Test { 31 class TransportFeedbackAdapterTest : public ::testing::Test {
32 public: 32 public:
33 TransportFeedbackAdapterTest() 33 TransportFeedbackAdapterTest()
34 : clock_(0), bitrate_controller_(this), target_bitrate_bps_(0) {} 34 : clock_(0), bitrate_controller_(this), target_bitrate_bps_(0) {}
35 35
36 virtual ~TransportFeedbackAdapterTest() {} 36 virtual ~TransportFeedbackAdapterTest() {}
37 37
38 virtual void SetUp() { 38 virtual void SetUp() {
39 adapter_.reset(new TransportFeedbackAdapter(&clock_, &bitrate_controller_)); 39 adapter_.reset(
40 new TransportFeedbackAdapter(nullptr, &clock_, &bitrate_controller_));
40 adapter_->InitBwe(); 41 adapter_->InitBwe();
41 adapter_->SetStartBitrate(300000); 42 adapter_->SetStartBitrate(300000);
42 } 43 }
43 44
44 virtual void TearDown() { adapter_.reset(); } 45 virtual void TearDown() { adapter_.reset(); }
45 46
46 protected: 47 protected:
47 // Proxy class used since TransportFeedbackAdapter will own the instance 48 // Proxy class used since TransportFeedbackAdapter will own the instance
48 // passed at construction. 49 // passed at construction.
49 class MockBitrateControllerAdapter : public MockBitrateController { 50 class MockBitrateControllerAdapter : public MockBitrateController {
(...skipping 381 matching lines...) Expand 10 before | Expand all | Expand 10 after
431 EXPECT_TRUE(feedback.get() != nullptr); 432 EXPECT_TRUE(feedback.get() != nullptr);
432 adapter_->OnTransportFeedback(*feedback.get()); 433 adapter_->OnTransportFeedback(*feedback.get());
433 clock_.AdvanceTimeMilliseconds(50); 434 clock_.AdvanceTimeMilliseconds(50);
434 ++seq_num; 435 ++seq_num;
435 } 436 }
436 EXPECT_GT(target_bitrate_bps_, 0u); 437 EXPECT_GT(target_bitrate_bps_, 0u);
437 } 438 }
438 439
439 } // namespace test 440 } // namespace test
440 } // namespace webrtc 441 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698