From 65e7d4cf20c32f6ecd0dcb9c0db0ac94ed8aafbe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Peter=20Bostr=C3=B6m?= Date: Mon, 26 Oct 2015 17:39:35 +0100 Subject: [PATCH] Remove CanCreateAndDestroyManyVideoStreams. This test was used to verify that VideoEngine handles were handed back correctly. This is no longer applicable. BUG=webrtc:1695 R=mflodman@webrtc.org Review URL: https://codereview.webrtc.org/1425673002 . Cr-Commit-Position: refs/heads/master@{#10412} --- webrtc/video/end_to_end_tests.cc | 34 -------------------------------- 1 file changed, 34 deletions(-) diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc index d889290da8..31ec7d5a76 100644 --- a/webrtc/video/end_to_end_tests.cc +++ b/webrtc/video/end_to_end_tests.cc @@ -3125,40 +3125,6 @@ TEST_F(EndToEndTest, NewReceiveStreamsRespectNetworkDown) { DestroyStreams(); } -// TODO(pbos): Remove this regression test when VideoEngine is no longer used as -// a backend. This is to test that we hand channels back properly. -TEST_F(EndToEndTest, CanCreateAndDestroyManyVideoStreams) { - test::NullTransport transport; - rtc::scoped_ptr call(Call::Create(Call::Config())); - test::FakeDecoder fake_decoder; - test::FakeEncoder fake_encoder(Clock::GetRealTimeClock()); - for (size_t i = 0; i < 100; ++i) { - VideoSendStream::Config send_config(&transport); - send_config.encoder_settings.encoder = &fake_encoder; - send_config.encoder_settings.payload_name = "FAKE"; - send_config.encoder_settings.payload_type = 123; - - VideoEncoderConfig encoder_config; - encoder_config.streams = test::CreateVideoStreams(1); - send_config.rtp.ssrcs.push_back(1); - VideoSendStream* send_stream = - call->CreateVideoSendStream(send_config, encoder_config); - call->DestroyVideoSendStream(send_stream); - - VideoReceiveStream::Config receive_config(&transport); - receive_config.rtp.remote_ssrc = 1; - receive_config.rtp.local_ssrc = kReceiverLocalSsrc; - VideoReceiveStream::Decoder decoder; - decoder.decoder = &fake_decoder; - decoder.payload_type = 123; - decoder.payload_name = "FAKE"; - receive_config.decoders.push_back(decoder); - VideoReceiveStream* receive_stream = - call->CreateVideoReceiveStream(receive_config); - call->DestroyVideoReceiveStream(receive_stream); - } -} - void VerifyEmptyNackConfig(const NackConfig& config) { EXPECT_EQ(0, config.rtp_history_ms) << "Enabling NACK requires rtcp-fb: nack negotiation.";