commit 72f52a1883f39e76cd571eddc26c2f26d594e417
author Niels Möller <nisse@webrtc.org> Tue Jun 26 09:14:11 2018
committer Commit Bot <commit-bot@chromium.org> Tue Jun 26 14:34:15 2018
tree 0b1a3b725a40b19d62f74652b6e9c361250b85fe
parent fe288eb687b8deb9eafcf156ac39793a57870c19

Delete unused copy constructors for VCMEncodedFrame and VCMFrameBuffer.

Bug: webrtc:9378
Change-Id: I742c7e2ca11f9c12d65add2bac9d7d19e09e3f14
Reviewed-on: https://webrtc-review.googlesource.com/85367
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23744}

modules/video_coding/encoded_frame.cc

diff --git a/modules/video_coding/encoded_frame.cc b/modules/video_coding/encoded_frame.cc
index cb9ff6c..a3d93b2 100644
--- a/modules/video_coding/encoded_frame.cc
+++ b/modules/video_coding/encoded_frame.cc
@@ -22,43 +22,6 @@
   _codecSpecificInfo.codecType = kVideoCodecUnknown;
 }
 
-VCMEncodedFrame::VCMEncodedFrame(const webrtc::EncodedImage& rhs)
-    : webrtc::EncodedImage(rhs),
-      _renderTimeMs(-1),
-      _payloadType(0),
-      _missingFrame(false),
-      _codec(kVideoCodecUnknown),
-      _rotation_set(false) {
-  _codecSpecificInfo.codecType = kVideoCodecUnknown;
-  _buffer = NULL;
-  _size = 0;
-  _length = 0;
-  if (rhs._buffer != NULL) {
-    VerifyAndAllocate(rhs._length +
-                      EncodedImage::GetBufferPaddingBytes(_codec));
-    memcpy(_buffer, rhs._buffer, rhs._length);
-  }
-}
-
-VCMEncodedFrame::VCMEncodedFrame(const VCMEncodedFrame& rhs)
-    : webrtc::EncodedImage(rhs),
-      _renderTimeMs(rhs._renderTimeMs),
-      _payloadType(rhs._payloadType),
-      _missingFrame(rhs._missingFrame),
-      _codecSpecificInfo(rhs._codecSpecificInfo),
-      _codec(rhs._codec),
-      _rotation_set(rhs._rotation_set) {
-  _buffer = NULL;
-  _size = 0;
-  _length = 0;
-  if (rhs._buffer != NULL) {
-    VerifyAndAllocate(rhs._length +
-                      EncodedImage::GetBufferPaddingBytes(_codec));
-    memcpy(_buffer, rhs._buffer, rhs._length);
-    _length = rhs._length;
-  }
-}
-
 VCMEncodedFrame::~VCMEncodedFrame() {
   Free();
 }

modules/video_coding/encoded_frame.h

diff --git a/modules/video_coding/encoded_frame.h b/modules/video_coding/encoded_frame.h
index 17c61d7..252dea4 100644
--- a/modules/video_coding/encoded_frame.h
+++ b/modules/video_coding/encoded_frame.h
@@ -24,8 +24,7 @@
 class VCMEncodedFrame : protected EncodedImage {
  public:
   VCMEncodedFrame();
-  explicit VCMEncodedFrame(const webrtc::EncodedImage& rhs);
-  VCMEncodedFrame(const VCMEncodedFrame& rhs);
+  VCMEncodedFrame(const VCMEncodedFrame&) = delete;
 
   ~VCMEncodedFrame();
   /**

modules/video_coding/frame_buffer.cc

diff --git a/modules/video_coding/frame_buffer.cc b/modules/video_coding/frame_buffer.cc
index 99abf43..b9241b9 100644
--- a/modules/video_coding/frame_buffer.cc
+++ b/modules/video_coding/frame_buffer.cc
@@ -25,16 +25,6 @@
 
 VCMFrameBuffer::~VCMFrameBuffer() {}
 
-VCMFrameBuffer::VCMFrameBuffer(const VCMFrameBuffer& rhs)
-    : VCMEncodedFrame(rhs),
-      _state(rhs._state),
-      _sessionInfo(),
-      _nackCount(rhs._nackCount),
-      _latestPacketTimeMs(rhs._latestPacketTimeMs) {
-  _sessionInfo = rhs._sessionInfo;
-  _sessionInfo.UpdateDataPointers(rhs._buffer, _buffer);
-}
-
 webrtc::FrameType VCMFrameBuffer::FrameType() const {
   return _sessionInfo.FrameType();
 }

modules/video_coding/frame_buffer.h

diff --git a/modules/video_coding/frame_buffer.h b/modules/video_coding/frame_buffer.h
index 66f338a..2604717 100644
--- a/modules/video_coding/frame_buffer.h
+++ b/modules/video_coding/frame_buffer.h
@@ -27,8 +27,6 @@
   VCMFrameBuffer();
   virtual ~VCMFrameBuffer();
 
-  VCMFrameBuffer(const VCMFrameBuffer& rhs);
-
   virtual void Reset();
 
   VCMFrameBufferEnum InsertPacket(const VCMPacket& packet,