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

Unified Diff: webrtc/call/call_perf_tests.cc

Issue 2774463003: Don't hardcode MediaType::ANY in FakeNetworkPipe. (Closed)
Patch Set: Update FakeCall::DeliverPacket, for consistency with Call::DeliverRtp. 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
« no previous file with comments | « webrtc/call/call.cc ('k') | webrtc/call/rampup_tests.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/call/call_perf_tests.cc
diff --git a/webrtc/call/call_perf_tests.cc b/webrtc/call/call_perf_tests.cc
index a421c24d56558fd29880f0074338b938b3a94a6e..9dcd6f394836820ab39fe143bb2bd5f8f597f88e 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());
@@ -342,12 +345,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.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698