Remove usage of VoENetwork from VoEWrapper and FakeWebRtcVoiceEngine.

BUG=webrtc:4690

Review-Url: https://codereview.webrtc.org/1934513002
Cr-Commit-Position: refs/heads/master@{#12566}
This commit is contained in:
solenberg 2016-04-29 09:05:28 -07:00 committed by Commit bot
parent 79e2842381
commit 05e61edd8f
4 changed files with 3 additions and 52 deletions

View File

@ -121,15 +121,13 @@ class FakeAudioProcessing : public webrtc::AudioProcessing {
class FakeWebRtcVoiceEngine
: public webrtc::VoEAudioProcessing,
public webrtc::VoEBase, public webrtc::VoECodec,
public webrtc::VoEHardware,
public webrtc::VoENetwork, public webrtc::VoERTP_RTCP,
public webrtc::VoEHardware, public webrtc::VoERTP_RTCP,
public webrtc::VoEVolumeControl {
public:
struct Channel {
Channel() {
memset(&send_codec, 0, sizeof(send_codec));
}
bool external_transport = false;
bool playout = false;
float volume_scale = 1.0f;
bool vad = false;
@ -146,8 +144,6 @@ class FakeWebRtcVoiceEngine
int associate_send_channel = -1;
std::vector<webrtc::CodecInst> recv_codecs;
webrtc::CodecInst send_codec;
webrtc::PacketTime last_rtp_packet_time;
std::list<std::string> packets;
int neteq_capacity = -1;
bool neteq_fast_accelerate = false;
};
@ -191,10 +187,6 @@ class FakeWebRtcVoiceEngine
int GetNACKMaxPackets(int channel) {
return channels_[channel]->nack_max_packets;
}
const webrtc::PacketTime& GetLastRtpPacketTime(int channel) {
RTC_DCHECK(channels_.find(channel) != channels_.end());
return channels_[channel]->last_rtp_packet_time;
}
int GetSendCNPayloadType(int channel, bool wideband) {
return (wideband) ?
channels_[channel]->cn16_type :
@ -455,40 +447,6 @@ class FakeWebRtcVoiceEngine
WEBRTC_STUB(EnableBuiltInNS, (bool enable));
bool BuiltInNSIsAvailable() const override { return false; }
// webrtc::VoENetwork
WEBRTC_FUNC(RegisterExternalTransport, (int channel,
webrtc::Transport& transport)) {
WEBRTC_CHECK_CHANNEL(channel);
channels_[channel]->external_transport = true;
return 0;
}
WEBRTC_FUNC(DeRegisterExternalTransport, (int channel)) {
WEBRTC_CHECK_CHANNEL(channel);
channels_[channel]->external_transport = false;
return 0;
}
WEBRTC_FUNC(ReceivedRTPPacket, (int channel, const void* data,
size_t length)) {
WEBRTC_CHECK_CHANNEL(channel);
if (!channels_[channel]->external_transport) return -1;
channels_[channel]->packets.push_back(
std::string(static_cast<const char*>(data), length));
return 0;
}
WEBRTC_FUNC(ReceivedRTPPacket, (int channel, const void* data,
size_t length,
const webrtc::PacketTime& packet_time)) {
WEBRTC_CHECK_CHANNEL(channel);
if (ReceivedRTPPacket(channel, data, length) == -1) {
return -1;
}
channels_[channel]->last_rtp_packet_time = packet_time;
return 0;
}
WEBRTC_STUB(ReceivedRTCPPacket, (int channel, const void* data,
size_t length));
// webrtc::VoERTP_RTCP
WEBRTC_FUNC(SetLocalSSRC, (int channel, unsigned int ssrc)) {
WEBRTC_CHECK_CHANNEL(channel);

View File

@ -74,15 +74,13 @@ class VoEWrapper {
public:
VoEWrapper()
: engine_(webrtc::VoiceEngine::Create()), processing_(engine_),
base_(engine_), codec_(engine_),
hw_(engine_), network_(engine_),
rtp_(engine_), volume_(engine_) {
base_(engine_), codec_(engine_), hw_(engine_), rtp_(engine_),
volume_(engine_) {
}
VoEWrapper(webrtc::VoEAudioProcessing* processing,
webrtc::VoEBase* base,
webrtc::VoECodec* codec,
webrtc::VoEHardware* hw,
webrtc::VoENetwork* network,
webrtc::VoERTP_RTCP* rtp,
webrtc::VoEVolumeControl* volume)
: engine_(NULL),
@ -90,7 +88,6 @@ class VoEWrapper {
base_(base),
codec_(codec),
hw_(hw),
network_(network),
rtp_(rtp),
volume_(volume) {
}
@ -100,7 +97,6 @@ class VoEWrapper {
webrtc::VoEBase* base() const { return base_.get(); }
webrtc::VoECodec* codec() const { return codec_.get(); }
webrtc::VoEHardware* hw() const { return hw_.get(); }
webrtc::VoENetwork* network() const { return network_.get(); }
webrtc::VoERTP_RTCP* rtp() const { return rtp_.get(); }
webrtc::VoEVolumeControl* volume() const { return volume_.get(); }
int error() { return base_->LastError(); }
@ -111,7 +107,6 @@ class VoEWrapper {
scoped_voe_ptr<webrtc::VoEBase> base_;
scoped_voe_ptr<webrtc::VoECodec> codec_;
scoped_voe_ptr<webrtc::VoEHardware> hw_;
scoped_voe_ptr<webrtc::VoENetwork> network_;
scoped_voe_ptr<webrtc::VoERTP_RTCP> rtp_;
scoped_voe_ptr<webrtc::VoEVolumeControl> volume_;
};

View File

@ -2268,7 +2268,6 @@ void WebRtcVoiceMediaChannel::OnPacketReceived(
call_->Receiver()->DeliverPacket(webrtc::MediaType::AUDIO,
packet->cdata(), packet->size(),
webrtc_packet_time);
if (delivery_result != webrtc::PacketReceiver::DELIVERY_UNKNOWN_SSRC) {
return;
}

View File

@ -58,7 +58,6 @@ class FakeVoEWrapper : public cricket::VoEWrapper {
engine, // base
engine, // codec
engine, // hw
engine, // network
engine, // rtp
engine) { // volume
}