diff --git a/webrtc/video/full_stack.cc b/webrtc/video/full_stack.cc index cb0ba55a38..cb97cd85bb 100644 --- a/webrtc/video/full_stack.cc +++ b/webrtc/video/full_stack.cc @@ -125,10 +125,6 @@ class VideoAnalyzer : public PacketReceiver, return receiver_->DeliverPacket(packet, length); } - virtual void PutFrame(const I420VideoFrame& video_frame) OVERRIDE { - ADD_FAILURE() << "PutFrame() should not have been called in this test."; - } - virtual void SwapFrame(I420VideoFrame* video_frame) OVERRIDE { I420VideoFrame* copy = NULL; { diff --git a/webrtc/video/video_send_stream.cc b/webrtc/video/video_send_stream.cc index 7cdb35339f..ced0b697ef 100644 --- a/webrtc/video/video_send_stream.cc +++ b/webrtc/video/video_send_stream.cc @@ -280,24 +280,12 @@ VideoSendStream::~VideoSendStream() { rtp_rtcp_->Release(); } -void VideoSendStream::PutFrame(const I420VideoFrame& frame) { - input_frame_.CopyFrame(frame); - SwapFrame(&input_frame_); -} - void VideoSendStream::SwapFrame(I420VideoFrame* frame) { - // TODO(pbos): Warn if frame is "too far" into the future, or too old. This - // would help detect if frame's being used without NTP. - // TO REVIEWER: Is there any good check for this? Should it be - // skipped? - if (frame != &input_frame_) - input_frame_.SwapFrame(frame); - // TODO(pbos): Local rendering should not be done on the capture thread. if (config_.local_renderer != NULL) - config_.local_renderer->RenderFrame(input_frame_, 0); + config_.local_renderer->RenderFrame(*frame, 0); - external_capture_->SwapFrame(&input_frame_); + external_capture_->SwapFrame(frame); } VideoSendStreamInput* VideoSendStream::Input() { return this; } diff --git a/webrtc/video/video_send_stream.h b/webrtc/video/video_send_stream.h index b8f5661fd0..a7e22676e4 100644 --- a/webrtc/video/video_send_stream.h +++ b/webrtc/video/video_send_stream.h @@ -57,7 +57,6 @@ class VideoSendStream : public webrtc::VideoSendStream, bool DeliverRtcp(const uint8_t* packet, size_t length); // From VideoSendStreamInput. - virtual void PutFrame(const I420VideoFrame& frame) OVERRIDE; virtual void SwapFrame(I420VideoFrame* frame) OVERRIDE; // From webrtc::VideoSendStream. @@ -69,7 +68,6 @@ class VideoSendStream : public webrtc::VideoSendStream, virtual std::string GetCName() OVERRIDE; private: - I420VideoFrame input_frame_; TransportAdapter transport_adapter_; EncodedFrameCallbackAdapter encoded_frame_proxy_; scoped_ptr codec_lock_; diff --git a/webrtc/video_send_stream.h b/webrtc/video_send_stream.h index 1a94121dcc..87c0dac999 100644 --- a/webrtc/video_send_stream.h +++ b/webrtc/video_send_stream.h @@ -29,7 +29,6 @@ class VideoSendStreamInput { // These methods do not lock internally and must be called sequentially. // If your application switches input sources synchronization must be done // externally to make sure that any old frames are not delivered concurrently. - virtual void PutFrame(const I420VideoFrame& video_frame) = 0; virtual void SwapFrame(I420VideoFrame* video_frame) = 0; protected: