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

Unified Diff: webrtc/api/rtpsenderreceiver_unittest.cc

Issue 2614263002: Remove BaseChannel's dependency on TransportController. (Closed)
Patch Set: cr comments Created 3 years, 11 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/api/rtpsenderreceiver_unittest.cc
diff --git a/webrtc/api/rtpsenderreceiver_unittest.cc b/webrtc/api/rtpsenderreceiver_unittest.cc
index a79b2976b6cf76f4d68464110c657c2cba4d8886..b6c51841bdd829c707d782e70882ea4b5fba2af3 100644
--- a/webrtc/api/rtpsenderreceiver_unittest.cc
+++ b/webrtc/api/rtpsenderreceiver_unittest.cc
@@ -64,12 +64,17 @@ class RtpSenderReceiverTest : public testing::Test {
channel_manager_.Init();
bool rtcp_enabled = false;
bool srtp_required = true;
+ cricket::TransportChannel* rtp_transport =
+ fake_transport_controller_.CreateTransportChannel(
+ cricket::CN_AUDIO, cricket::ICE_CANDIDATE_COMPONENT_RTP);
voice_channel_ = channel_manager_.CreateVoiceChannel(
- &fake_media_controller_, &fake_transport_controller_, cricket::CN_AUDIO,
- nullptr, rtcp_enabled, srtp_required, cricket::AudioOptions());
+ &fake_media_controller_, rtp_transport, nullptr, rtc::Thread::Current(),
+ cricket::CN_AUDIO, nullptr, rtcp_enabled, srtp_required,
+ cricket::AudioOptions());
video_channel_ = channel_manager_.CreateVideoChannel(
- &fake_media_controller_, &fake_transport_controller_, cricket::CN_VIDEO,
- nullptr, rtcp_enabled, srtp_required, cricket::VideoOptions());
+ &fake_media_controller_, rtp_transport, nullptr, rtc::Thread::Current(),
+ cricket::CN_VIDEO, nullptr, rtcp_enabled, srtp_required,
+ cricket::VideoOptions());
voice_media_channel_ = media_engine_->GetVoiceChannel(0);
video_media_channel_ = media_engine_->GetVideoChannel(0);
RTC_CHECK(voice_channel_);

Powered by Google App Engine
This is Rietveld 408576698