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

Unified Diff: webrtc/call/call_perf_tests.cc

Issue 2783853002: Reland of Don't hardcode MediaType::ANY in FakeNetworkPipe. (Closed)
Patch Set: Fix braces. Add comment on use of demuxer. Created 3 years, 9 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/call/call_perf_tests.cc
diff --git a/webrtc/call/call_perf_tests.cc b/webrtc/call/call_perf_tests.cc
index bd5ab0ff253a63d319a02d336432ac54bb53185c..d62bec14907969a83e7e0e3c3ef518b92fd63e8e 100644
--- a/webrtc/call/call_perf_tests.cc
+++ b/webrtc/call/call_perf_tests.cc
@@ -193,6 +193,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
audio_net_config.loss_percent = 5;
test::PacketTransport audio_send_transport(sender_call_.get(), &observer,
test::PacketTransport::kSender,
+ MediaType::AUDIO,
audio_net_config);
MediaTypePacketReceiver audio_receiver(receiver_call_->Receiver(),
MediaType::AUDIO);
@@ -200,6 +201,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
test::PacketTransport video_send_transport(sender_call_.get(), &observer,
test::PacketTransport::kSender,
+ MediaType::VIDEO,
FakeNetworkPipe::Config());
MediaTypePacketReceiver video_receiver(receiver_call_->Receiver(),
MediaType::VIDEO);
@@ -207,6 +209,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
test::PacketTransport receive_transport(
receiver_call_.get(), &observer, test::PacketTransport::kReceiver,
+ MediaType::VIDEO,
FakeNetworkPipe::Config());
receive_transport.SetReceiver(sender_call_->Receiver());
@@ -343,12 +346,14 @@ void CallPerfTest::TestCaptureNtpTime(const FakeNetworkPipe::Config& net_config,
private:
test::PacketTransport* CreateSendTransport(Call* sender_call) override {
return new test::PacketTransport(
- sender_call, this, test::PacketTransport::kSender, net_config_);
+ sender_call, this, test::PacketTransport::kSender, MediaType::VIDEO,
+ net_config_);
}
test::PacketTransport* CreateReceiveTransport() override {
return new test::PacketTransport(
- nullptr, this, test::PacketTransport::kReceiver, net_config_);
+ nullptr, this, test::PacketTransport::kReceiver, MediaType::VIDEO,
+ net_config_);
}
void OnFrame(const VideoFrame& video_frame) override {
« no previous file with comments | « webrtc/call/call.cc ('k') | webrtc/call/rampup_tests.h » ('j') | webrtc/test/direct_transport.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698