Add VideoFrame::CreateZeroInitializedFrame

media/base/video_frame_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/video_frame.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/format_macros.h"
 #include "base/memory/aligned_memory.h"
@@ skipping 140 matching lines @@
   VideoFrame::HashFrameForTesting(&context, frame);
   base::MD5Final(&digest, &context);
   EXPECT_EQ(MD5DigestToBase16(digest), "911991d51438ad2e1a40ed5f6fc7c796");
 
   // Test an empty frame.
   frame = VideoFrame::CreateEOSFrame();
   EXPECT_TRUE(
       frame->metadata()->IsTrue(VideoFrameMetadata::END_OF_STREAM));
 }
 
+TEST(VideoFrame, CreateZeroInitializedFrame) {
+  const int kWidth = 2;
+  const int kHeight = 2;
+  const base::TimeDelta kTimestamp = base::TimeDelta::FromMicroseconds(1337);
+
+  // Create a YV12 Video Frame.
+  gfx::Size size(kWidth, kHeight);
+  scoped_refptr<media::VideoFrame> frame =
+      VideoFrame::CreateZeroInitializedFrame(media::PIXEL_FORMAT_YV12, size,
+                                             gfx::Rect(size), size, kTimestamp);
+  ASSERT_TRUE(frame.get());
+  EXPECT_TRUE(frame->IsMappable());
+
+  // Verify that frame is initialized with zeros.
+  for (size_t i = 0; i < VideoFrame::NumPlanes(frame->format()); ++i)

[mcasas, 2015/09/05 01:29:25]

nit: please add a TODO to extend this part here
to check all contents -- not just the first byte ;)
(I know you got it from another test).

[emircan, 2015/09/05 01:39:50]

Done.

+    EXPECT_EQ(0, frame->data(i)[0]);
+}
+
 TEST(VideoFrame, CreateBlackFrame) {
   const int kWidth = 2;
   const int kHeight = 2;
   const uint8 kExpectedYRow[] = { 0, 0 };
   const uint8 kExpectedUVRow[] = { 128 };
 
   scoped_refptr<media::VideoFrame> frame =
       VideoFrame::CreateBlackFrame(gfx::Size(kWidth, kHeight));
   ASSERT_TRUE(frame.get());
   EXPECT_TRUE(frame->IsMappable());
@@ skipping 283 matching lines @@
 
   for (int i = 0; i < VideoFrameMetadata::NUM_KEYS; ++i) {
     const VideoFrameMetadata::Key key = static_cast<VideoFrameMetadata::Key>(i);
     int value = -1;
     EXPECT_TRUE(result.GetInteger(key, &value));
     EXPECT_EQ(i, value);
   }
 }
 
 }  // namespace media