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

Unified Diff: webrtc/call/call_perf_tests.cc

Issue 2794243002: Making FakeNetworkPipe demux audio and video packets. (Closed)
Patch Set: fixing android Created 3 years, 8 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 bfb11d03038f776bee9d18cad5daeb979b170ebb..7ade5283fea4225cf01e7d56498268d4d1c43f54 100644
--- a/webrtc/call/call_perf_tests.cc
+++ b/webrtc/call/call_perf_tests.cc
@@ -191,26 +191,37 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
FakeNetworkPipe::Config audio_net_config;
audio_net_config.queue_delay_ms = 500;
audio_net_config.loss_percent = 5;
+
+ std::map<uint8_t, MediaType> audio_pt_map;
+ std::map<uint8_t, MediaType> video_pt_map;
+ std::copy_if(std::begin(payload_type_map_), std::end(payload_type_map_),
+ std::inserter(audio_pt_map, audio_pt_map.end()),
+ [](const std::pair<const uint8_t, MediaType>& pair) {
+ return pair.second == MediaType::AUDIO;
+ });
+ std::copy_if(std::begin(payload_type_map_), std::end(payload_type_map_),
+ std::inserter(video_pt_map, video_pt_map.end()),
+ [](const std::pair<const uint8_t, MediaType>& pair) {
+ return pair.second == MediaType::VIDEO;
+ });
+
test::PacketTransport audio_send_transport(sender_call_.get(), &observer,
test::PacketTransport::kSender,
- MediaType::AUDIO,
- audio_net_config);
+ audio_pt_map, audio_net_config);
MediaTypePacketReceiver audio_receiver(receiver_call_->Receiver(),
MediaType::AUDIO);
audio_send_transport.SetReceiver(&audio_receiver);
- test::PacketTransport video_send_transport(sender_call_.get(), &observer,
- test::PacketTransport::kSender,
- MediaType::VIDEO,
- FakeNetworkPipe::Config());
+ test::PacketTransport video_send_transport(
+ sender_call_.get(), &observer, test::PacketTransport::kSender,
+ video_pt_map, FakeNetworkPipe::Config());
MediaTypePacketReceiver video_receiver(receiver_call_->Receiver(),
MediaType::VIDEO);
video_send_transport.SetReceiver(&video_receiver);
test::PacketTransport receive_transport(
receiver_call_.get(), &observer, test::PacketTransport::kReceiver,
- MediaType::VIDEO,
- FakeNetworkPipe::Config());
+ payload_type_map_, FakeNetworkPipe::Config());
receive_transport.SetReceiver(sender_call_->Receiver());
test::FakeDecoder fake_decoder;
@@ -222,7 +233,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
audio_send_config.voe_channel_id = send_channel_id;
audio_send_config.rtp.ssrc = kAudioSendSsrc;
audio_send_config.send_codec_spec.codec_inst =
- CodecInst{103, "ISAC", 16000, 480, 1, 32000};
+ CodecInst{kAudioSendPayloadType, "ISAC", 16000, 480, 1, 32000};
AudioSendStream* audio_send_stream =
sender_call_->CreateAudioSendStream(audio_send_config);
@@ -244,7 +255,7 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
audio_recv_config.voe_channel_id = recv_channel_id;
audio_recv_config.sync_group = kSyncGroup;
audio_recv_config.decoder_factory = decoder_factory_;
- audio_recv_config.decoder_map = {{103, {"ISAC", 16000, 1}}};
+ audio_recv_config.decoder_map = {{kAudioSendPayloadType, {"ISAC", 16000, 1}}};
AudioReceiveStream* audio_receive_stream;
@@ -345,15 +356,15 @@ 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, MediaType::VIDEO,
- net_config_);
+ return new test::PacketTransport(sender_call, this,
+ test::PacketTransport::kSender,
+ payload_type_map_, net_config_);
}
test::PacketTransport* CreateReceiveTransport() override {
- return new test::PacketTransport(
- nullptr, this, test::PacketTransport::kReceiver, MediaType::VIDEO,
- net_config_);
+ return new test::PacketTransport(nullptr, this,
+ test::PacketTransport::kReceiver,
+ payload_type_map_, net_config_);
}
void OnFrame(const VideoFrame& video_frame) override {

Powered by Google App Engine
This is Rietveld 408576698