Bug 1497992 - Remove VideoReceiver::Reset; r=pehrsons

This ends up calling VCMReceiver::Reset() which resets the
state of the VCMJitterBuffer. We no longer use VCMJitterBuffer,
which is the old jitter buffer implementation, so this code
no longer has any effect and can be removed.

Differential Revision: https://phabricator.services.mozilla.com/D14185

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Dan Minor 2018-12-12 17:57:38 +00:00
parent 56891ff038
commit 5e0f0c2fb0
5 changed files with 0 additions and 9 deletions

View File

@ -414,8 +414,6 @@ class VideoCodingModule : public Module {
EncodedImageCallback* post_encode_callback) = 0;
// Releases pending decode calls, permitting faster thread shutdown.
virtual void TriggerDecoderShutdown() = 0;
// resets underlying objects
virtual void Reset() = 0;
};
} // namespace webrtc

View File

@ -256,7 +256,6 @@ class VideoCodingModuleImpl : public VideoCodingModule {
}
void TriggerDecoderShutdown() override { receiver_.TriggerDecoderShutdown(); }
void Reset() override {receiver_.Reset(); }
private:
rtc::ThreadChecker construction_thread_;

View File

@ -189,7 +189,6 @@ class VideoReceiver : public Module {
void Process() override;
void TriggerDecoderShutdown();
void Reset();
protected:
int32_t Decode(const webrtc::VCMEncodedFrame& frame)

View File

@ -218,10 +218,6 @@ void VideoReceiver::TriggerDecoderShutdown() {
_receiver.TriggerDecoderShutdown();
}
void VideoReceiver::Reset() {
_receiver.Reset();
}
// Decode next frame, blocking.
// Should be called as often as possible to get the most out of the decoder.
int32_t VideoReceiver::Decode(uint16_t maxWaitTimeMs) {

View File

@ -170,7 +170,6 @@ void VideoReceiveStream::Start() {
RTC_DCHECK_CALLED_SEQUENTIALLY(&worker_sequence_checker_);
if (decode_thread_.IsRunning())
return;
video_receiver_.Reset();
bool protected_by_fec = config_.rtp.protected_by_flexfec ||
rtp_video_stream_receiver_.IsUlpfecEnabled();