Cleanup of webrtc::VideoFrame.

webrtc/common_video/libyuv/libyuv_unittest.cc

 /*
  *  Copyright (c) 2012 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.
  */
 
@@ skipping 93 matching lines @@
 
 void TestLibYuv::SetUp() {
   const std::string input_file_name = webrtc::test::ResourcePath("foreman_cif",
                                                                  "yuv");
   source_file_  = fopen(input_file_name.c_str(), "rb");
   ASSERT_TRUE(source_file_ != NULL) << "Cannot read file: "<<
                                        input_file_name << "\n";
 
   EXPECT_EQ(frame_length_,
             fread(orig_buffer_.get(), 1, frame_length_, source_file_));
-  EXPECT_EQ(0, orig_frame_.CreateFrame(orig_buffer_.get(),
-                                       orig_buffer_.get() + size_y_,
-                                       orig_buffer_.get() +
-                                       size_y_ + size_uv_,
-                                       width_, height_,
-                                       width_, (width_ + 1) / 2,
-                                       (width_ + 1) / 2));
+  orig_frame_.CreateFrame(orig_buffer_.get(),
+                          orig_buffer_.get() + size_y_,
+                          orig_buffer_.get() +
+                          size_y_ + size_uv_,
+                          width_, height_,
+                          width_, (width_ + 1) / 2,
+                          (width_ + 1) / 2,
+                          kVideoRotation_0);
 }
 
 void TestLibYuv::TearDown() {
   if (source_file_ != NULL) {
     ASSERT_EQ(0, fclose(source_file_));
   }
   source_file_ = NULL;
 }
 
 TEST_F(TestLibYuv, ConvertSanityTest) {
   // TODO(mikhal)
 }
 
 TEST_F(TestLibYuv, ConvertTest) {
   // Reading YUV frame - testing on the first frame of the foreman sequence
   int j = 0;
   std::string output_file_name = webrtc::test::OutputPath() +
                                  "LibYuvTest_conversion.yuv";
   FILE*  output_file = fopen(output_file_name.c_str(), "wb");
   ASSERT_TRUE(output_file != NULL);
 
   double psnr = 0.0;
 
   VideoFrame res_i420_frame;
-  EXPECT_EQ(0, res_i420_frame.CreateEmptyFrame(width_, height_, width_,
+  res_i420_frame.CreateEmptyFrame(width_, height_, width_,
                                                (width_ + 1) / 2,
-                                               (width_ + 1) / 2));
+                                               (width_ + 1) / 2);
   printf("\nConvert #%d I420 <-> I420 \n", j);
   rtc::scoped_ptr<uint8_t[]> out_i420_buffer(new uint8_t[frame_length_]);
   EXPECT_EQ(0, ConvertFromI420(orig_frame_, kI420, 0,
                                out_i420_buffer.get()));
   EXPECT_EQ(0, ConvertToI420(kI420, out_i420_buffer.get(), 0, 0, width_,
                              height_, 0, kVideoRotation_0, &res_i420_frame));
 
   if (PrintVideoFrame(res_i420_frame, output_file) < 0) {
     return;
   }
@@ skipping 37 matching lines @@
 
   printf("\nConvert #%d I420 <-> YV12\n", j);
   rtc::scoped_ptr<uint8_t[]> outYV120Buffer(new uint8_t[frame_length_]);
   rtc::scoped_ptr<uint8_t[]> res_i420_buffer(new uint8_t[frame_length_]);
   VideoFrame yv12_frame;
   EXPECT_EQ(0, ConvertFromI420(orig_frame_, kYV12, 0, outYV120Buffer.get()));
   yv12_frame.CreateFrame(outYV120Buffer.get(),
                          outYV120Buffer.get() + size_y_,
                          outYV120Buffer.get() + size_y_ + size_uv_,
                          width_, height_,
-                         width_, (width_ + 1) / 2, (width_ + 1) / 2);
+                         width_, (width_ + 1) / 2, (width_ + 1) / 2,
+                         kVideoRotation_0);
   EXPECT_EQ(0, ConvertFromYV12(yv12_frame, kI420, 0, res_i420_buffer.get()));
   if (fwrite(res_i420_buffer.get(), 1, frame_length_, output_file) !=
       frame_length_) {
     return;
   }
 
   ConvertToI420(kI420, res_i420_buffer.get(), 0, 0, width_, height_, 0,
                 kVideoRotation_0, &res_i420_frame);
   psnr = I420PSNR(&orig_frame_, &res_i420_frame);
   EXPECT_EQ(48.0, psnr);
@@ skipping 58 matching lines @@
                                  "LibYuvTest_conversion.yuv";
   FILE*  output_file = fopen(output_file_name.c_str(), "wb");
   ASSERT_TRUE(output_file != NULL);
 
   double psnr = 0.0;
 
   VideoFrame res_i420_frame;
   int stride_y = 0;
   int stride_uv = 0;
   Calc16ByteAlignedStride(width_, &stride_y, &stride_uv);
-  EXPECT_EQ(0, res_i420_frame.CreateEmptyFrame(width_, height_,
-                                               stride_y, stride_uv, stride_uv));
+  res_i420_frame.CreateEmptyFrame(width_, height_,
+                                  stride_y, stride_uv, stride_uv);
   rtc::scoped_ptr<uint8_t[]> out_i420_buffer(new uint8_t[frame_length_]);
   EXPECT_EQ(0, ConvertFromI420(orig_frame_, kI420, 0,
                                out_i420_buffer.get()));
   EXPECT_EQ(0, ConvertToI420(kI420, out_i420_buffer.get(), 0, 0, width_,
                              height_, 0, kVideoRotation_0, &res_i420_frame));
 
   if (PrintVideoFrame(res_i420_frame, output_file) < 0) {
     return;
   }
   psnr = I420PSNR(&orig_frame_, &res_i420_frame);
   EXPECT_EQ(48.0, psnr);
 }
 
 
 TEST_F(TestLibYuv, RotateTest) {
   // Use ConvertToI420 for multiple roatations - see that nothing breaks, all
   // memory is properly allocated and end result is equal to the starting point.
   VideoFrame rotated_res_i420_frame;
   int rotated_width = height_;
   int rotated_height = width_;
   int stride_y;
   int stride_uv;
   Calc16ByteAlignedStride(rotated_width, &stride_y, &stride_uv);
-  EXPECT_EQ(0, rotated_res_i420_frame.CreateEmptyFrame(rotated_width,
-                                                       rotated_height,
-                                                       stride_y,
-                                                       stride_uv,
-                                                       stride_uv));
+  rotated_res_i420_frame.CreateEmptyFrame(rotated_width,
+                                          rotated_height,
+                                          stride_y,
+                                          stride_uv,
+                                          stride_uv);
   EXPECT_EQ(0, ConvertToI420(kI420, orig_buffer_.get(), 0, 0, width_, height_,
                              0, kVideoRotation_90, &rotated_res_i420_frame));
   EXPECT_EQ(0, ConvertToI420(kI420, orig_buffer_.get(), 0, 0, width_, height_,
                              0, kVideoRotation_270, &rotated_res_i420_frame));
-  EXPECT_EQ(0, rotated_res_i420_frame.CreateEmptyFrame(width_, height_,
-                                                       width_, (width_ + 1) / 2,
-                                                       (width_ + 1) / 2));
+  rotated_res_i420_frame.CreateEmptyFrame(width_, height_,
+                                          width_, (width_ + 1) / 2,
+                                          (width_ + 1) / 2);
   EXPECT_EQ(0, ConvertToI420(kI420, orig_buffer_.get(), 0, 0, width_, height_,
                              0, kVideoRotation_180, &rotated_res_i420_frame));
 }
 
 TEST_F(TestLibYuv, alignment) {
   int value = 0x3FF;  // 1023
   EXPECT_EQ(0x400, AlignInt(value, 128));  // Low 7 bits are zero.
   EXPECT_EQ(0x400, AlignInt(value, 64));  // Low 6 bits are zero.
   EXPECT_EQ(0x400, AlignInt(value, 32));  // Low 5 bits are zero.
 }
 
 TEST_F(TestLibYuv, StrideAlignment) {
   int stride_y = 0;
   int stride_uv = 0;
   int width = 52;
   Calc16ByteAlignedStride(width, &stride_y, &stride_uv);
   EXPECT_EQ(64, stride_y);
   EXPECT_EQ(32, stride_uv);
   width = 128;
   Calc16ByteAlignedStride(width, &stride_y, &stride_uv);
   EXPECT_EQ(128, stride_y);
   EXPECT_EQ(64, stride_uv);
   width = 127;
   Calc16ByteAlignedStride(width, &stride_y, &stride_uv);
   EXPECT_EQ(128, stride_y);
   EXPECT_EQ(64, stride_uv);
 }
 
 }  // namespace webrtc