commit 6d057976e9539f722c1e6e33f65c36bc7d5dc8d9
author Asa Persson <asapersson@webrtc.org> Fri Nov 19 13:07:10 2021
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Nov 23 11:10:04 2021
tree 842476060f3bf84825802c8633f94c8d60186fd8
parent f47a72416840d0de587d9b0bc99ed06f53573cfb

StatsEndToEndTests: Remove unneeded fake renderer in tests.

Bug: none
Change-Id: I1f14fa2e606dde38113560c32640fb6b2996ea14
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/238428
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Åsa Persson <asapersson@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35404}

video/end_to_end_tests/stats_tests.cc

diff --git a/video/end_to_end_tests/stats_tests.cc b/video/end_to_end_tests/stats_tests.cc
index 06821e9..10591b4 100644
--- a/video/end_to_end_tests/stats_tests.cc
+++ b/video/end_to_end_tests/stats_tests.cc
@@ -48,24 +48,13 @@
   static const int kStartBitrateBps = 3000000;
   static const int kExpectedRenderDelayMs = 20;
 
-  class ReceiveStreamRenderer : public rtc::VideoSinkInterface<VideoFrame> {
-   public:
-    ReceiveStreamRenderer() {}
-
-   private:
-    void OnFrame(const VideoFrame& video_frame) override {}
-  };
-
   class StatsObserver : public test::EndToEndTest {
    public:
     StatsObserver()
-        : EndToEndTest(kLongTimeoutMs),
-          encoder_factory_([]() {
+        : EndToEndTest(kLongTimeoutMs), encoder_factory_([]() {
             return std::make_unique<test::DelayedEncoder>(
                 Clock::GetRealTimeClock(), 10);
-          }),
-          send_stream_(nullptr),
-          expected_send_ssrcs_() {}
+          }) {}
 
    private:
     Action OnSendRtp(const uint8_t* packet, size_t length) override {
@@ -274,7 +263,6 @@
         expected_receive_ssrcs_.push_back(
             (*receive_configs)[i].rtp.remote_ssrc);
         (*receive_configs)[i].render_delay_ms = kExpectedRenderDelayMs;
-        (*receive_configs)[i].renderer = &receive_stream_renderer_;
         (*receive_configs)[i].rtp.nack.rtp_history_ms = kNackRtpHistoryMs;
 
         (*receive_configs)[i].rtp.rtx_ssrc = kSendRtxSsrcs[i];
@@ -342,14 +330,13 @@
     std::vector<VideoReceiveStream*> receive_streams_;
     std::map<std::string, bool> receive_stats_filled_;
 
-    VideoSendStream* send_stream_;
+    VideoSendStream* send_stream_ = nullptr;
     std::map<std::string, bool> send_stats_filled_;
 
     std::vector<uint32_t> expected_receive_ssrcs_;
     std::set<uint32_t> expected_send_ssrcs_;
 
     rtc::Event check_stats_event_;
-    ReceiveStreamRenderer receive_stream_renderer_;
     TaskQueueBase* task_queue_ = nullptr;
   } test;
 
@@ -666,7 +653,6 @@
         VideoEncoderConfig* encoder_config) override {
       send_config->rtp.nack.rtp_history_ms = kNackRtpHistoryMs;
       (*receive_configs)[0].rtp.nack.rtp_history_ms = kNackRtpHistoryMs;
-      (*receive_configs)[0].renderer = &fake_renderer_;
     }
 
     void OnVideoStreamsCreated(
@@ -682,7 +668,6 @@
       EXPECT_TRUE(Wait()) << "Timed out waiting for packet to be NACKed.";
     }
 
-    test::FakeVideoRenderer fake_renderer_;
     Mutex mutex_;
     uint64_t sent_rtp_packets_ RTC_GUARDED_BY(&mutex_) = 0;
     uint16_t dropped_rtp_packet_ RTC_GUARDED_BY(&mutex_) = 0;