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

Unified Diff: webrtc/video/encoder_state_feedback_unittest.cc

Issue 1917793002: Remove SendPacer from ViEEncoder (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Addressed code review comments. Created 4 years, 8 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/video/encoder_state_feedback_unittest.cc
diff --git a/webrtc/video/encoder_state_feedback_unittest.cc b/webrtc/video/encoder_state_feedback_unittest.cc
index 3b7ef29d00ed34302641ac1c80115238402f18d9..a4c7932c283222ec86f46a67e6fca13408f09c93 100644
--- a/webrtc/video/encoder_state_feedback_unittest.cc
+++ b/webrtc/video/encoder_state_feedback_unittest.cc
@@ -25,15 +25,13 @@ namespace webrtc {
class MockVieEncoder : public ViEEncoder {
public:
- explicit MockVieEncoder(ProcessThread* process_thread, PacedSender* pacer)
+ explicit MockVieEncoder(ProcessThread* process_thread)
: ViEEncoder(1,
std::vector<uint32_t>(),
process_thread,
nullptr,
nullptr,
nullptr,
- pacer,
- nullptr,
nullptr) {}
~MockVieEncoder() {}
@@ -49,12 +47,7 @@ TEST(VieKeyRequestTest, CreateAndTriggerRequests) {
static const uint32_t kSsrc = 1234;
NiceMock<MockProcessThread> process_thread;
PacketRouter router;
- PacedSender pacer(Clock::GetRealTimeClock(), &router,
- BitrateController::kDefaultStartBitrateKbps,
- PacedSender::kDefaultPaceMultiplier *
- BitrateController::kDefaultStartBitrateKbps,
- 0);
- MockVieEncoder encoder(&process_thread, &pacer);
+ MockVieEncoder encoder(&process_thread);
EncoderStateFeedback encoder_state_feedback;
encoder_state_feedback.Init(std::vector<uint32_t>(1, kSsrc), &encoder);

Powered by Google App Engine
This is Rietveld 408576698