Revert of New static method I420Buffer::SetToBlack.

webrtc/media/engine/webrtcvideoframe_unittest.cc

 /*
  *  Copyright (c) 2011 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #include <string.h>
 
 #include <memory>
 
 #include "webrtc/media/base/videoframe_unittest.h"
 #include "webrtc/media/engine/webrtcvideoframe.h"
 #include "webrtc/test/fake_texture_frame.h"
 
 namespace {
 
 class WebRtcVideoTestFrame : public cricket::WebRtcVideoFrame {
  public:
-  WebRtcVideoTestFrame() {}
-  WebRtcVideoTestFrame(
-      const rtc::scoped_refptr<webrtc::VideoFrameBuffer>& buffer,
-      int64_t time_stamp_ns,
-      webrtc::VideoRotation rotation)
-    : WebRtcVideoFrame(buffer, time_stamp_ns, rotation) {}
-
   // The ApplyRotationToFrame test needs this as a public method.
   using cricket::WebRtcVideoFrame::set_rotation;
 
   virtual VideoFrame* CreateEmptyFrame(int w,
                                        int h,
                                        int64_t time_stamp) const override {
-    rtc::scoped_refptr<webrtc::I420Buffer> buffer(
-        new rtc::RefCountedObject<webrtc::I420Buffer>(w, h));
-    buffer->SetToBlack();
-    return new WebRtcVideoTestFrame(
-        buffer, time_stamp, webrtc::kVideoRotation_0);
+    WebRtcVideoTestFrame* frame = new WebRtcVideoTestFrame();
+    frame->InitToBlack(w, h, time_stamp);
+    return frame;
   }
 };
 
 }  // namespace
 
 class WebRtcVideoFrameTest : public VideoFrameTest<cricket::WebRtcVideoFrame> {
  public:
   WebRtcVideoFrameTest() {
   }
 
@@ skipping 96 matching lines @@
 TEST_WEBRTCVIDEOFRAME(ConstructI4201Pixel)
 TEST_WEBRTCVIDEOFRAME(ConstructI4205Pixel)
 // TODO(juberti): WebRtcVideoFrame does not support horizontal crop.
 // Re-evaluate once it supports 3 independent planes, since we might want to
 // just Init normally and then crop by adjusting pointers.
 // TEST_WEBRTCVIDEOFRAME(ConstructI420CropHorizontal)
 TEST_WEBRTCVIDEOFRAME(ConstructI420CropVertical)
 // TODO(juberti): WebRtcVideoFrame is not currently refcounted.
 // TEST_WEBRTCVIDEOFRAME(ConstructCopy)
 // TEST_WEBRTCVIDEOFRAME(ConstructCopyIsRef)
+TEST_WEBRTCVIDEOFRAME(ConstructBlack)
 // TODO(fbarchard): Implement Jpeg
 // TEST_WEBRTCVIDEOFRAME(ConstructMjpgI420)
 TEST_WEBRTCVIDEOFRAME(ConstructMjpgI422)
 // TEST_WEBRTCVIDEOFRAME(ConstructMjpgI444)
 // TEST_WEBRTCVIDEOFRAME(ConstructMjpgI411)
 // TEST_WEBRTCVIDEOFRAME(ConstructMjpgI400)
 // TEST_WEBRTCVIDEOFRAME(ValidateMjpgI420)
 // TEST_WEBRTCVIDEOFRAME(ValidateMjpgI422)
 // TEST_WEBRTCVIDEOFRAME(ValidateMjpgI444)
 // TEST_WEBRTCVIDEOFRAME(ValidateMjpgI411)
@@ skipping 164 matching lines @@
 
   // Claim the frame 2 needs to be rotated for another 270 degree. The output
   // from frame 2 rotation should be the same as frame 1.
   applied90->set_rotation(webrtc::kVideoRotation_270);
   const cricket::VideoFrame* applied360 =
       applied90->GetCopyWithRotationApplied();
   EXPECT_TRUE(applied360);
   EXPECT_EQ(applied360->rotation(), webrtc::kVideoRotation_0);
   EXPECT_TRUE(IsEqual(applied0, *applied360, 0));
 }