diff options
Diffstat (limited to 'webrtc/test/channel_transport')
-rw-r--r-- | webrtc/test/channel_transport/channel_transport.cc | 6 | ||||
-rw-r--r-- | webrtc/test/channel_transport/channel_transport.h (renamed from webrtc/test/channel_transport/include/channel_transport.h) | 6 | ||||
-rw-r--r-- | webrtc/test/channel_transport/udp_socket2_manager_win.cc | 21 | ||||
-rw-r--r-- | webrtc/test/channel_transport/udp_socket2_manager_win.h | 6 | ||||
-rw-r--r-- | webrtc/test/channel_transport/udp_socket2_win.cc | 10 | ||||
-rw-r--r-- | webrtc/test/channel_transport/udp_socket_manager_posix.cc | 27 | ||||
-rw-r--r-- | webrtc/test/channel_transport/udp_socket_manager_posix.h | 4 |
7 files changed, 32 insertions, 48 deletions
diff --git a/webrtc/test/channel_transport/channel_transport.cc b/webrtc/test/channel_transport/channel_transport.cc index 25eb59d887..38eefe54a2 100644 --- a/webrtc/test/channel_transport/channel_transport.cc +++ b/webrtc/test/channel_transport/channel_transport.cc @@ -8,7 +8,7 @@ * be found in the AUTHORS file in the root of the source tree. */ -#include "webrtc/test/channel_transport/include/channel_transport.h" +#include "webrtc/test/channel_transport/channel_transport.h" #include <stdio.h> @@ -16,7 +16,6 @@ #include "testing/gtest/include/gtest/gtest.h" #endif #include "webrtc/test/channel_transport/udp_transport.h" -#include "webrtc/video_engine/vie_defines.h" #include "webrtc/voice_engine/include/voe_network.h" #if defined(WEBRTC_ANDROID) || defined(WEBRTC_IOS) @@ -66,10 +65,11 @@ void VoiceChannelTransport::IncomingRTCPPacket( } int VoiceChannelTransport::SetLocalReceiver(uint16_t rtp_port) { + static const int kNumReceiveSocketBuffers = 500; int return_value = socket_transport_->InitializeReceiveSockets(this, rtp_port); if (return_value == 0) { - return socket_transport_->StartReceiving(kViENumReceiveSocketBuffers); + return socket_transport_->StartReceiving(kNumReceiveSocketBuffers); } return return_value; } diff --git a/webrtc/test/channel_transport/include/channel_transport.h b/webrtc/test/channel_transport/channel_transport.h index 8b84517de0..bab7c59181 100644 --- a/webrtc/test/channel_transport/include/channel_transport.h +++ b/webrtc/test/channel_transport/channel_transport.h @@ -8,8 +8,8 @@ * be found in the AUTHORS file in the root of the source tree. */ -#ifndef WEBRTC_TEST_CHANNEL_TRANSPORT_INCLUDE_CHANNEL_TRANSPORT_H_ -#define WEBRTC_TEST_CHANNEL_TRANSPORT_INCLUDE_CHANNEL_TRANSPORT_H_ +#ifndef WEBRTC_TEST_CHANNEL_TRANSPORT_CHANNEL_TRANSPORT_H_ +#define WEBRTC_TEST_CHANNEL_TRANSPORT_CHANNEL_TRANSPORT_H_ #include "webrtc/test/channel_transport/udp_transport.h" @@ -53,4 +53,4 @@ class VoiceChannelTransport : public UdpTransportData { } // namespace test } // namespace webrtc -#endif // WEBRTC_TEST_CHANNEL_TRANSPORT_INCLUDE_CHANNEL_TRANSPORT_H_ +#endif // WEBRTC_TEST_CHANNEL_TRANSPORT_CHANNEL_TRANSPORT_H_ diff --git a/webrtc/test/channel_transport/udp_socket2_manager_win.cc b/webrtc/test/channel_transport/udp_socket2_manager_win.cc index 5a11abbd6e..9f40350287 100644 --- a/webrtc/test/channel_transport/udp_socket2_manager_win.cc +++ b/webrtc/test/channel_transport/udp_socket2_manager_win.cc @@ -520,8 +520,8 @@ int32_t UdpSocket2WorkerWindows::_numOfWorkers = 0; UdpSocket2WorkerWindows::UdpSocket2WorkerWindows(HANDLE ioCompletionHandle) : _ioCompletionHandle(ioCompletionHandle), - _init(false) -{ + _pThread(Run, this, "UdpSocket2ManagerWindows_thread"), + _init(false) { _workerNumber = _numOfWorkers++; WEBRTC_TRACE(kTraceMemory, kTraceTransport, -1, "UdpSocket2WorkerWindows created"); @@ -537,10 +537,9 @@ bool UdpSocket2WorkerWindows::Start() { WEBRTC_TRACE(kTraceStateInfo, kTraceTransport, -1, "Start UdpSocket2WorkerWindows"); - if (!_pThread->Start()) - return false; + _pThread.Start(); - _pThread->SetPriority(kRealtimePriority); + _pThread.SetPriority(rtc::kRealtimePriority); return true; } @@ -548,18 +547,14 @@ bool UdpSocket2WorkerWindows::Stop() { WEBRTC_TRACE(kTraceStateInfo, kTraceTransport, -1, "Stop UdpSocket2WorkerWindows"); - return _pThread->Stop(); + _pThread.Stop(); + return true; } int32_t UdpSocket2WorkerWindows::Init() { - if(!_init) - { - const char* threadName = "UdpSocket2ManagerWindows_thread"; - _pThread = ThreadWrapper::CreateThread(Run, this, threadName); - _init = true; - } - return 0; + _init = true; + return 0; } bool UdpSocket2WorkerWindows::Run(void* obj) diff --git a/webrtc/test/channel_transport/udp_socket2_manager_win.h b/webrtc/test/channel_transport/udp_socket2_manager_win.h index c6af03a702..e762dccd0d 100644 --- a/webrtc/test/channel_transport/udp_socket2_manager_win.h +++ b/webrtc/test/channel_transport/udp_socket2_manager_win.h @@ -17,7 +17,7 @@ #include "webrtc/system_wrappers/include/atomic32.h" #include "webrtc/system_wrappers/include/critical_section_wrapper.h" #include "webrtc/system_wrappers/include/event_wrapper.h" -#include "webrtc/system_wrappers/include/thread_wrapper.h" +#include "webrtc/base/platform_thread.h" #include "webrtc/test/channel_transport/udp_socket2_win.h" #include "webrtc/test/channel_transport/udp_socket_manager_wrapper.h" #include "webrtc/test/channel_transport/udp_transport.h" @@ -47,7 +47,7 @@ struct PerIoContext { int fromLen; // Should be set to true if the I/O context was passed to the system by // a thread not controlled by the socket implementation. - bool ioInitiatedByThreadWrapper; + bool ioInitiatedByPlatformThread; // TODO (hellner): Not used. Delete it. PerIoContext* pNextFree; }; @@ -105,7 +105,7 @@ protected: bool Process(); private: HANDLE _ioCompletionHandle; - rtc::scoped_ptr<ThreadWrapper> _pThread; + rtc::PlatformThread _pThread; static int32_t _numOfWorkers; int32_t _workerNumber; volatile bool _stop; diff --git a/webrtc/test/channel_transport/udp_socket2_win.cc b/webrtc/test/channel_transport/udp_socket2_win.cc index 4c63dc938d..adeb46a9d2 100644 --- a/webrtc/test/channel_transport/udp_socket2_win.cc +++ b/webrtc/test/channel_transport/udp_socket2_win.cc @@ -432,13 +432,13 @@ void UdpSocket2Windows::IOCompleted(PerIoContext* pIOContext, if(pIOContext == NULL || error == ERROR_OPERATION_ABORTED) { if ((pIOContext != NULL) && - !pIOContext->ioInitiatedByThreadWrapper && + !pIOContext->ioInitiatedByPlatformThread && (error == ERROR_OPERATION_ABORTED) && (pIOContext->ioOperation == OP_READ) && _outstandingCallsDisabled) { - // !pIOContext->initiatedIOByThreadWrapper indicate that the I/O - // was not initiated by a ThreadWrapper thread. + // !pIOContext->initiatedIOByPlatformThread indicate that the I/O + // was not initiated by a PlatformThread thread. // This may happen if the thread that initiated receiving (e.g. // by calling StartListen())) is deleted before any packets have // been received. @@ -519,7 +519,7 @@ void UdpSocket2Windows::IOCompleted(PerIoContext* pIOContext, { // The PerIoContext was posted by a thread controlled by the socket // implementation. - pIOContext->ioInitiatedByThreadWrapper = true; + pIOContext->ioInitiatedByPlatformThread = true; } OutstandingCallCompleted(); return; @@ -546,7 +546,7 @@ int32_t UdpSocket2Windows::PostRecv() } // This function may have been called by thread not controlled by the socket // implementation. - pIoContext->ioInitiatedByThreadWrapper = false; + pIoContext->ioInitiatedByPlatformThread = false; return PostRecv(pIoContext); } diff --git a/webrtc/test/channel_transport/udp_socket_manager_posix.cc b/webrtc/test/channel_transport/udp_socket_manager_posix.cc index 145efcbc58..6b1a466bf2 100644 --- a/webrtc/test/channel_transport/udp_socket_manager_posix.cc +++ b/webrtc/test/channel_transport/udp_socket_manager_posix.cc @@ -184,12 +184,11 @@ bool UdpSocketManagerPosix::RemoveSocket(UdpSocketWrapper* s) return retVal; } - UdpSocketManagerPosixImpl::UdpSocketManagerPosixImpl() -{ - _critSectList = CriticalSectionWrapper::CreateCriticalSection(); - _thread = ThreadWrapper::CreateThread(UdpSocketManagerPosixImpl::Run, this, - "UdpSocketManagerPosixImplThread"); + : _thread(UdpSocketManagerPosixImpl::Run, + this, + "UdpSocketManagerPosixImplThread"), + _critSectList(CriticalSectionWrapper::CreateCriticalSection()) { FD_ZERO(&_readFds); WEBRTC_TRACE(kTraceMemory, kTraceTransport, -1, "UdpSocketManagerPosix created"); @@ -220,29 +219,19 @@ UdpSocketManagerPosixImpl::~UdpSocketManagerPosixImpl() bool UdpSocketManagerPosixImpl::Start() { - if (!_thread) - { - return false; - } - WEBRTC_TRACE(kTraceStateInfo, kTraceTransport, -1, "Start UdpSocketManagerPosix"); - if (!_thread->Start()) - return false; - _thread->SetPriority(kRealtimePriority); + _thread.Start(); + _thread.SetPriority(rtc::kRealtimePriority); return true; } bool UdpSocketManagerPosixImpl::Stop() { - if (!_thread) - { - return true; - } - WEBRTC_TRACE(kTraceStateInfo, kTraceTransport, -1, "Stop UdpSocketManagerPosix"); - return _thread->Stop(); + _thread.Stop(); + return true; } bool UdpSocketManagerPosixImpl::Process() diff --git a/webrtc/test/channel_transport/udp_socket_manager_posix.h b/webrtc/test/channel_transport/udp_socket_manager_posix.h index 64156fd20f..45e55af99a 100644 --- a/webrtc/test/channel_transport/udp_socket_manager_posix.h +++ b/webrtc/test/channel_transport/udp_socket_manager_posix.h @@ -17,8 +17,8 @@ #include <list> #include <map> +#include "webrtc/base/platform_thread.h" #include "webrtc/system_wrappers/include/critical_section_wrapper.h" -#include "webrtc/system_wrappers/include/thread_wrapper.h" #include "webrtc/test/channel_transport/udp_socket_manager_wrapper.h" #include "webrtc/test/channel_transport/udp_socket_wrapper.h" @@ -75,7 +75,7 @@ protected: private: typedef std::list<UdpSocketWrapper*> SocketList; typedef std::list<SOCKET> FdList; - rtc::scoped_ptr<ThreadWrapper> _thread; + rtc::PlatformThread _thread; CriticalSectionWrapper* _critSectList; fd_set _readFds; |