Delete webrtc::VideoFrame::CopyFrame.

BUG=webrtc:5682

Review-Url: https://codereview.webrtc.org/2371363003
Cr-Commit-Position: refs/heads/master@{#14550}

webrtc/common_video/i420_video_frame_unittest.cc

@@ -84,12 +84,6 @@ TEST(TestVideoFrame, InitialValues) {
   EXPECT_EQ(kVideoRotation_0, frame.rotation());
 }
 
-TEST(TestVideoFrame, CopiesInitialFrameWithoutCrashing) {
-  VideoFrame frame;
-  VideoFrame frame2;
-  frame2.CopyFrame(frame);
-}
-
 TEST(TestVideoFrame, WidthHeightValues) {
   VideoFrame frame(I420Buffer::Create(10, 10, 10, 14, 90),
                    webrtc::kVideoRotation_0,
@@ -104,44 +98,6 @@ TEST(TestVideoFrame, WidthHeightValues) {
   EXPECT_EQ(789, frame.render_time_ms());
 }
 
-TEST(TestVideoFrame, CopyFrame) {
-  int stride_y = 15;
-  int stride_u = 10;
-  int stride_v = 10;
-  int width = 15;
-  int height = 15;
-  // Copy frame.
-  VideoFrame small_frame;
-  const int kSizeY = 400;
-  const int kSizeU = 100;
-  const int kSizeV = 100;
-  const VideoRotation kRotation = kVideoRotation_270;
-  uint8_t buffer_y[kSizeY];
-  uint8_t buffer_u[kSizeU];
-  uint8_t buffer_v[kSizeV];
-  memset(buffer_y, 16, kSizeY);
-  memset(buffer_u, 8, kSizeU);
-  memset(buffer_v, 4, kSizeV);
-  VideoFrame big_frame;
-  big_frame.CreateFrame(buffer_y, buffer_u, buffer_v,
-                        width + 5, height + 5, stride_y + 5,
-                        stride_u, stride_v, kRotation);
-  // Frame of smaller dimensions.
-  small_frame.CopyFrame(big_frame);
-  EXPECT_TRUE(test::FramesEqual(small_frame, big_frame));
-  EXPECT_EQ(kRotation, small_frame.rotation());
-
-  // Frame of larger dimensions.
-  rtc::scoped_refptr<I420Buffer> buffer =
-      I420Buffer::Create(width, height, stride_y, stride_u, stride_v);
-  memset(buffer->MutableDataY(), 1, width * height);
-  memset(buffer->MutableDataU(), 2, ((height + 1) / 2) * stride_u);
-  memset(buffer->MutableDataV(), 3, ((height + 1) / 2) * stride_u);
-  VideoFrame other_frame(buffer, 0, 0, webrtc::kVideoRotation_0);
-  big_frame.CopyFrame(other_frame);
-  EXPECT_TRUE(test::FramesEqual(other_frame, big_frame));
-}
-
 TEST(TestVideoFrame, ShallowCopy) {
   uint32_t timestamp = 1;
   int64_t ntp_time_ms = 2;

webrtc/common_video/video_frame.cc

@@ -93,15 +93,6 @@ void VideoFrame::CreateFrame(const uint8_t* buffer,
               stride_uv, stride_uv, rotation);
 }
 
-void VideoFrame::CopyFrame(const VideoFrame& videoFrame) {
-  ShallowCopy(videoFrame);
-
-  // If backed by a plain memory buffer, create a new, non-shared, copy.
-  if (video_frame_buffer_ && !video_frame_buffer_->native_handle()) {
-    video_frame_buffer_ = I420Buffer::Copy(video_frame_buffer_);
-  }
-}
-
 void VideoFrame::ShallowCopy(const VideoFrame& videoFrame) {
   video_frame_buffer_ = videoFrame.video_frame_buffer();
   timestamp_rtp_ = videoFrame.timestamp_rtp_;

webrtc/modules/video_coding/codecs/vp8/test/vp8_impl_unittest.cc

@@ -110,7 +110,7 @@ bool Vp8UnitTestDecodeCompleteCallback::DecodeComplete() {
 }
 
 int Vp8UnitTestDecodeCompleteCallback::Decoded(VideoFrame& image) {
-  decoded_frame_->CopyFrame(image);
+  decoded_frame_->ShallowCopy(image);
   decode_complete = true;
   return 0;
 }

webrtc/modules/video_processing/test/video_processing_unittest.cc

@@ -250,9 +250,10 @@ void TestSize(const VideoFrame& source_frame,
 
   // Scale |resampled_source_frame| back to the source scale.
   VideoFrame resampled_source_frame;
-  resampled_source_frame.CopyFrame(*out_frame);
+  resampled_source_frame.ShallowCopy(*out_frame);
   // Compute PSNR against the cropped source frame and check expectation.
-  PreprocessFrameAndVerify(resampled_source_frame, cropped_source.width(),
+  PreprocessFrameAndVerify(resampled_source_frame,
+                           cropped_source.width(),
                            cropped_source.height(), vpm, out_frame);
   WriteProcessedFrameForVisualInspection(resampled_source_frame, *out_frame);
 

webrtc/video/video_quality_test.cc

@@ -482,8 +482,8 @@ class VideoAnalyzer : public PacketReceiver,
 
     rtc::CritScope crit(&comparison_lock_);
     if (comparisons_.size() < kMaxComparisons) {
-      reference_copy.CopyFrame(reference);
+      reference_copy.ShallowCopy(reference);
-      render_copy.CopyFrame(render);
+      render_copy.ShallowCopy(render);
     } else {
       // Copy the time to ensure that delay calculations can still be made.
       reference_copy.set_ntp_time_ms(reference.ntp_time_ms());

webrtc/video_frame.h

@@ -70,13 +70,6 @@ class VideoFrame {
                   int height,
                   VideoRotation rotation);
 
-  // Deep copy frame: If required size is bigger than allocated one, new
-  // buffers of adequate size will be allocated.
-  // TODO(nisse): Should be deleted in the cricket::VideoFrame and
-  // webrtc::VideoFrame merge. Instead, use I420Buffer::Copy to make a copy of
-  // the pixel data, and use the constructor to wrap it into a VideoFrame.
-  void CopyFrame(const VideoFrame& videoFrame);
-
   // Creates a shallow copy of |videoFrame|, i.e, the this object will retain a
   // reference to the video buffer also retained by |videoFrame|.
   // TODO(nisse): Deprecated. Should be deleted in the cricket::VideoFrame and