From c6e16e3d91d8a02ec5844317e484b8a7ff771072 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20Bostr=C3=B6m?= Date: Fri, 5 Feb 2016 14:15:53 +0100 Subject: [PATCH] Use a delayed encoder in GetStats test. Guarantees seeing non-zero CpuOveruseMetrics stats. BUG= R=stefan@webrtc.org Review URL: https://codereview.webrtc.org/1670763005 . Cr-Commit-Position: refs/heads/master@{#11504} --- webrtc/video/end_to_end_tests.cc | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc index 5385450298..28d371a5af 100644 --- a/webrtc/video/end_to_end_tests.cc +++ b/webrtc/video/end_to_end_tests.cc @@ -2554,6 +2554,7 @@ TEST_F(EndToEndTest, GetStats) { public: StatsObserver() : EndToEndTest(kLongTimeoutMs), + encoder_(Clock::GetRealTimeClock(), 10), send_stream_(nullptr), expected_send_ssrcs_(), check_stats_event_(false, false) {} @@ -2751,6 +2752,9 @@ TEST_F(EndToEndTest, GetStats) { (*receive_configs)[i].rtp.remote_ssrc); (*receive_configs)[i].render_delay_ms = kExpectedRenderDelayMs; } + // Use a delayed encoder to make sure we see CpuOveruseMetrics stats that + // are non-zero. + send_config->encoder_settings.encoder = &encoder_; } size_t GetNumVideoStreams() const override { return kNumSsrcs; } @@ -2804,6 +2808,7 @@ TEST_F(EndToEndTest, GetStats) { } } + test::DelayedEncoder encoder_; std::vector receive_streams_; std::map receive_stats_filled_;