From 79cbe69274429eebd3ca84d5314946490b867925 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Erik=20Spr=C3=A5ng?= Date: Tue, 13 Apr 2021 16:30:24 +0200 Subject: [PATCH] Removes incorrect test expectation. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As of https://source.chromium.org/chromium/_/webrtc/src.git/+/1e4d4fdf88370637318d38cbf62c731d04fa3e56 we no longer expect an InitEncode on deativation of a layer. Bug: webrtc:12540 Change-Id: I10d447d90d1019258f662caf7f6e649d63d6927a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/215076 Reviewed-by: Ilya Nikolaevskiy Commit-Queue: Erik Språng Cr-Commit-Position: refs/heads/master@{#33709} --- video/video_send_stream_tests.cc | 7 ------- 1 file changed, 7 deletions(-) diff --git a/video/video_send_stream_tests.cc b/video/video_send_stream_tests.cc index 47bcaab1a2..3ee97a9202 100644 --- a/video/video_send_stream_tests.cc +++ b/video/video_send_stream_tests.cc @@ -2272,13 +2272,6 @@ TEST_F(VideoSendStreamTest, VideoSendStreamUpdateActiveSimulcastLayers) { GetVideoSendStream()->ReconfigureVideoEncoder( GetVideoEncoderConfig()->Copy()); }); - // TODO(bugs.webrtc.org/8807): Currently we require a hard reconfiguration to - // update the VideoBitrateAllocator and BitrateAllocator of which layers are - // active. Once the change is made for a "soft" reconfiguration we can remove - // the expecation for an encoder init. We can also test that bitrate changes - // when just updating individual active layers, which should change the - // bitrate set to the video encoder. - EXPECT_TRUE(encoder.WaitForEncoderInit()); EXPECT_TRUE(encoder.WaitBitrateChanged(true)); // Turning off both simulcast layers should trigger a bitrate change of 0.