commit 46c4e609396acdc8874aa18017c201c60dc19a5d
author Artem Titov <titovartem@webrtc.org> Fri Aug 17 12:26:54 2018
committer Commit Bot <commit-bot@chromium.org> Mon Aug 20 07:23:41 2018
tree 3f39cadbe65708c045a3b281cf30117204d94363
parent c7ca7b19c7148c4c72fc6f7968d3328afe02ef64

Introduce SimulatedNetworkReceiverInterface.

Introduce SimulatedNetworkReceiverInterface and switch DirectTransport
on this interface. Also switch part of related users on
DefaultNetworkSimulationConfig.

This two changes united into single CL to prevent work duplication.
Most changes were done because of stop including fake_network_pipe.h
into direct_transport.h, so splitting this into 2 CLs will require
first fix all imports of fake_network_pipe.h and then replace them
on new API imports again.

Bug: webrtc:9630
Change-Id: I87d4a6ff1bab72d04a9871a40441f4fbe028f4e6
Reviewed-on: https://webrtc-review.googlesource.com/94762
Commit-Queue: Patrik Höglund <phoglund@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24336}

test/direct_transport.cc

diff --git a/test/direct_transport.cc b/test/direct_transport.cc
index 93221a6..351210e 100644
--- a/test/direct_transport.cc
+++ b/test/direct_transport.cc
@@ -11,6 +11,7 @@
 
 #include "absl/memory/memory.h"
 #include "call/call.h"
+#include "call/fake_network_pipe.h"
 #include "modules/rtp_rtcp/include/rtp_header_parser.h"
 #include "system_wrappers/include/clock.h"
 #include "test/single_threaded_task_queue.h"
@@ -40,13 +41,13 @@
     Call* send_call,
     const std::map<uint8_t, MediaType>& payload_type_map)
     : DirectTransport(task_queue,
-                      FakeNetworkPipe::Config(),
+                      DefaultNetworkSimulationConfig(),
                       send_call,
                       payload_type_map) {}
 
 DirectTransport::DirectTransport(
     SingleThreadedTaskQueueForTesting* task_queue,
-    const FakeNetworkPipe::Config& config,
+    const DefaultNetworkSimulationConfig& config,
     Call* send_call,
     const std::map<uint8_t, MediaType>& payload_type_map)
     : send_call_(send_call),
@@ -59,7 +60,7 @@
 
 DirectTransport::DirectTransport(
     SingleThreadedTaskQueueForTesting* task_queue,
-    std::unique_ptr<FakeNetworkPipe> pipe,
+    std::unique_ptr<SimulatedPacketReceiverInterface> pipe,
     Call* send_call,
     const std::map<uint8_t, MediaType>& payload_type_map)
     : send_call_(send_call),
@@ -81,7 +82,7 @@
   fake_network_->SetClockOffset(offset_ms);
 }
 
-void DirectTransport::SetConfig(const FakeNetworkPipe::Config& config) {
+void DirectTransport::SetConfig(const DefaultNetworkSimulationConfig& config) {
   fake_network_->SetConfig(config);
 }