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

Unified Diff: webrtc/call/call_perf_tests.cc

Issue 2870713003: Delete helper class MediaTypePacketReceiver. (Closed)
Patch Set: Created 3 years, 7 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 | « no previous file | no next file » | 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 1e6f28d340cdf1c73f196315aca3258ab32ad357..3b1de73e65147cefbcdad5e75b6e5f86991687f4 100644
--- a/webrtc/call/call_perf_tests.cc
+++ b/webrtc/call/call_perf_tests.cc
@@ -168,27 +168,6 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
VideoRtcpAndSyncObserver observer(Clock::GetRealTimeClock());
- // Helper class to ensure we deliver correct media_type to the receiving call.
- class MediaTypePacketReceiver : public PacketReceiver {
- public:
- MediaTypePacketReceiver(PacketReceiver* packet_receiver,
- MediaType media_type)
- : packet_receiver_(packet_receiver), media_type_(media_type) {}
-
- DeliveryStatus DeliverPacket(MediaType media_type,
- const uint8_t* packet,
- size_t length,
- const PacketTime& packet_time) override {
- return packet_receiver_->DeliverPacket(media_type_, packet, length,
- packet_time);
- }
- private:
- PacketReceiver* packet_receiver_;
- const MediaType media_type_;
-
- RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(MediaTypePacketReceiver);
- };
-
FakeNetworkPipe::Config audio_net_config;
audio_net_config.queue_delay_ms = 500;
audio_net_config.loss_percent = 5;
@@ -209,16 +188,12 @@ void CallPerfTest::TestAudioVideoSync(FecMode fec,
test::PacketTransport audio_send_transport(sender_call_.get(), &observer,
test::PacketTransport::kSender,
audio_pt_map, audio_net_config);
- MediaTypePacketReceiver audio_receiver(receiver_call_->Receiver(),
- MediaType::AUDIO);
- audio_send_transport.SetReceiver(&audio_receiver);
+ audio_send_transport.SetReceiver(receiver_call_->Receiver());
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);
+ video_send_transport.SetReceiver(receiver_call_->Receiver());
test::PacketTransport receive_transport(
receiver_call_.get(), &observer, test::PacketTransport::kReceiver,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698