diff options
Diffstat (limited to 'webrtc')
-rw-r--r-- | webrtc/p2p/base/p2ptransportchannel.cc | 3 | ||||
-rw-r--r-- | webrtc/p2p/base/p2ptransportchannel.h | 2 | ||||
-rw-r--r-- | webrtc/p2p/base/port.cc | 4 | ||||
-rw-r--r-- | webrtc/p2p/base/port.h | 6 | ||||
-rw-r--r-- | webrtc/p2p/base/port_unittest.cc | 4 | ||||
-rw-r--r-- | webrtc/p2p/base/portinterface.h | 2 | ||||
-rw-r--r-- | webrtc/p2p/base/relayport.cc | 2 | ||||
-rw-r--r-- | webrtc/p2p/base/relayport.h | 7 | ||||
-rw-r--r-- | webrtc/p2p/base/stunport.cc | 2 | ||||
-rw-r--r-- | webrtc/p2p/base/tcpport.cc | 7 | ||||
-rw-r--r-- | webrtc/p2p/base/tcpport.h | 3 | ||||
-rw-r--r-- | webrtc/p2p/base/turnport.cc | 7 | ||||
-rw-r--r-- | webrtc/p2p/base/turnport.h | 2 |
13 files changed, 12 insertions, 39 deletions
diff --git a/webrtc/p2p/base/p2ptransportchannel.cc b/webrtc/p2p/base/p2ptransportchannel.cc index 952cfab747..c58a235f2d 100644 --- a/webrtc/p2p/base/p2ptransportchannel.cc +++ b/webrtc/p2p/base/p2ptransportchannel.cc @@ -1424,7 +1424,8 @@ void P2PTransportChannel::OnReadPacket(Connection* connection, } } -void P2PTransportChannel::OnSentPacket(const rtc::SentPacket& sent_packet) { +void P2PTransportChannel::OnSentPacket(PortInterface* port, + const rtc::SentPacket& sent_packet) { ASSERT(worker_thread_ == rtc::Thread::Current()); SignalSentPacket(this, sent_packet); diff --git a/webrtc/p2p/base/p2ptransportchannel.h b/webrtc/p2p/base/p2ptransportchannel.h index f2e9315343..3927b17659 100644 --- a/webrtc/p2p/base/p2ptransportchannel.h +++ b/webrtc/p2p/base/p2ptransportchannel.h @@ -232,7 +232,7 @@ class P2PTransportChannel : public TransportChannelImpl, void OnConnectionStateChange(Connection* connection); void OnReadPacket(Connection *connection, const char *data, size_t len, const rtc::PacketTime& packet_time); - void OnSentPacket(const rtc::SentPacket& sent_packet); + void OnSentPacket(PortInterface* port, const rtc::SentPacket& sent_packet); void OnReadyToSend(Connection* connection); void OnConnectionDestroyed(Connection *connection); diff --git a/webrtc/p2p/base/port.cc b/webrtc/p2p/base/port.cc index 9dd5c83fed..f6a30097f8 100644 --- a/webrtc/p2p/base/port.cc +++ b/webrtc/p2p/base/port.cc @@ -310,6 +310,10 @@ void Port::OnReadPacket( } } +void Port::OnSentPacket(const rtc::SentPacket& sent_packet) { + PortInterface::SignalSentPacket(this, sent_packet); +} + void Port::OnReadyToSend() { AddressMap::iterator iter = connections_.begin(); for (; iter != connections_.end(); ++iter) { diff --git a/webrtc/p2p/base/port.h b/webrtc/p2p/base/port.h index 436b1e7faa..57608b5b14 100644 --- a/webrtc/p2p/base/port.h +++ b/webrtc/p2p/base/port.h @@ -280,11 +280,7 @@ class Port : public PortInterface, public rtc::MessageHandler, const std::string& remote_ufrag); // Called when a packet has been sent to the socket. - // This is made pure virtual to notify subclasses of Port that they MUST - // listen to AsyncPacketSocket::SignalSentPacket and then call - // PortInterface::OnSentPacket. - virtual void OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet) = 0; + void OnSentPacket(const rtc::SentPacket& sent_packet); // Called when the socket is currently able to send. void OnReadyToSend(); diff --git a/webrtc/p2p/base/port_unittest.cc b/webrtc/p2p/base/port_unittest.cc index 449021ad9f..34806d9a79 100644 --- a/webrtc/p2p/base/port_unittest.cc +++ b/webrtc/p2p/base/port_unittest.cc @@ -204,10 +204,6 @@ class TestPort : public Port { } private: - void OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet) { - PortInterface::SignalSentPacket(sent_packet); - } rtc::scoped_ptr<ByteBuffer> last_stun_buf_; rtc::scoped_ptr<IceMessage> last_stun_msg_; int type_preference_ = 0; diff --git a/webrtc/p2p/base/portinterface.h b/webrtc/p2p/base/portinterface.h index e83879f3b7..d1c371d356 100644 --- a/webrtc/p2p/base/portinterface.h +++ b/webrtc/p2p/base/portinterface.h @@ -116,7 +116,7 @@ class PortInterface { const rtc::SocketAddress&> SignalReadPacket; // Emitted each time a packet is sent on this port. - sigslot::signal1<const rtc::SentPacket&> SignalSentPacket; + sigslot::signal2<PortInterface*, const rtc::SentPacket&> SignalSentPacket; virtual std::string ToString() const = 0; diff --git a/webrtc/p2p/base/relayport.cc b/webrtc/p2p/base/relayport.cc index 19883a3121..88adcf2f88 100644 --- a/webrtc/p2p/base/relayport.cc +++ b/webrtc/p2p/base/relayport.cc @@ -754,7 +754,7 @@ void RelayEntry::OnReadPacket( void RelayEntry::OnSentPacket(rtc::AsyncPacketSocket* socket, const rtc::SentPacket& sent_packet) { - port_->OnSentPacket(socket, sent_packet); + port_->OnSentPacket(sent_packet); } void RelayEntry::OnReadyToSend(rtc::AsyncPacketSocket* socket) { diff --git a/webrtc/p2p/base/relayport.h b/webrtc/p2p/base/relayport.h index fdbf6e6f98..4b74b91ade 100644 --- a/webrtc/p2p/base/relayport.h +++ b/webrtc/p2p/base/relayport.h @@ -29,7 +29,7 @@ class RelayConnection; // is created. The RelayEntry will try to reach the remote destination // by connecting to all available server addresses in a pre defined // order with a small delay in between. When a connection is -// successful all other connection attempts are aborted. +// successful all other connection attemts are aborted. class RelayPort : public Port { public: typedef std::pair<rtc::Socket::Option, int> OptionValue; @@ -96,11 +96,6 @@ class RelayPort : public Port { ProtocolType proto, const rtc::PacketTime& packet_time); - // The OnSentPacket callback is left empty here since they are handled by - // RelayEntry. - void OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet) override {} - private: friend class RelayEntry; diff --git a/webrtc/p2p/base/stunport.cc b/webrtc/p2p/base/stunport.cc index 8f37dd5218..67cf789da8 100644 --- a/webrtc/p2p/base/stunport.cc +++ b/webrtc/p2p/base/stunport.cc @@ -340,7 +340,7 @@ void UDPPort::OnReadPacket(rtc::AsyncPacketSocket* socket, void UDPPort::OnSentPacket(rtc::AsyncPacketSocket* socket, const rtc::SentPacket& sent_packet) { - PortInterface::SignalSentPacket(sent_packet); + Port::OnSentPacket(sent_packet); } void UDPPort::OnReadyToSend(rtc::AsyncPacketSocket* socket) { diff --git a/webrtc/p2p/base/tcpport.cc b/webrtc/p2p/base/tcpport.cc index 6a64174f00..23afc2f031 100644 --- a/webrtc/p2p/base/tcpport.cc +++ b/webrtc/p2p/base/tcpport.cc @@ -258,7 +258,6 @@ void TCPPort::OnNewConnection(rtc::AsyncPacketSocket* socket, incoming.socket = new_socket; incoming.socket->SignalReadPacket.connect(this, &TCPPort::OnReadPacket); incoming.socket->SignalReadyToSend.connect(this, &TCPPort::OnReadyToSend); - incoming.socket->SignalSentPacket.connect(this, &TCPPort::OnSentPacket); LOG_J(LS_VERBOSE, this) << "Accepted connection from " << incoming.addr.ToSensitiveString(); @@ -287,12 +286,6 @@ void TCPPort::OnReadPacket(rtc::AsyncPacketSocket* socket, Port::OnReadPacket(data, size, remote_addr, PROTO_TCP); } -void TCPPort::OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet) { - ASSERT(socket == socket_); - PortInterface::SignalSentPacket(sent_packet); -} - void TCPPort::OnReadyToSend(rtc::AsyncPacketSocket* socket) { Port::OnReadyToSend(); } diff --git a/webrtc/p2p/base/tcpport.h b/webrtc/p2p/base/tcpport.h index b298a17e9c..568dc65f3d 100644 --- a/webrtc/p2p/base/tcpport.h +++ b/webrtc/p2p/base/tcpport.h @@ -96,9 +96,6 @@ class TCPPort : public Port { const rtc::SocketAddress& remote_addr, const rtc::PacketTime& packet_time); - void OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet) override; - void OnReadyToSend(rtc::AsyncPacketSocket* socket); void OnAddressReady(rtc::AsyncPacketSocket* socket, diff --git a/webrtc/p2p/base/turnport.cc b/webrtc/p2p/base/turnport.cc index 5ed93dd1d8..022207aa5b 100644 --- a/webrtc/p2p/base/turnport.cc +++ b/webrtc/p2p/base/turnport.cc @@ -351,8 +351,6 @@ bool TurnPort::CreateTurnClientSocket() { socket_->SignalReadyToSend.connect(this, &TurnPort::OnReadyToSend); - socket_->SignalSentPacket.connect(this, &TurnPort::OnSentPacket); - // TCP port is ready to send stun requests after the socket is connected, // while UDP port is ready to do so once the socket is created. if (server_address_.proto == PROTO_TCP) { @@ -584,11 +582,6 @@ void TurnPort::OnReadPacket( } } -void TurnPort::OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet) { - PortInterface::SignalSentPacket(sent_packet); -} - void TurnPort::OnReadyToSend(rtc::AsyncPacketSocket* socket) { if (ready()) { Port::OnReadyToSend(); diff --git a/webrtc/p2p/base/turnport.h b/webrtc/p2p/base/turnport.h index 4d83806a37..a19f67632e 100644 --- a/webrtc/p2p/base/turnport.h +++ b/webrtc/p2p/base/turnport.h @@ -106,8 +106,6 @@ class TurnPort : public Port { const rtc::SocketAddress& remote_addr, const rtc::PacketTime& packet_time); - virtual void OnSentPacket(rtc::AsyncPacketSocket* socket, - const rtc::SentPacket& sent_packet); virtual void OnReadyToSend(rtc::AsyncPacketSocket* socket); virtual bool SupportsProtocol(const std::string& protocol) const { // Turn port only connects to UDP candidates. |