commit f82109cb4d638826f8c795a9bcf41a51c258ba2d
author magjed@webrtc.org <magjed@webrtc.org> Wed Mar 04 11:30:29 2015 +0000
committer magjed@webrtc.org <magjed@webrtc.org> Wed Mar 04 11:30:48 2015 +0000
tree b78bc6c1d7f6c471d003f77f17fc2f21681757fa
parent 487afc704d5c6a2519f2decdb47130c7b3424992

Initialize memory in I420VideoFrame unittest

Previously, when CreateEmptyFrame was called with a smaller size than before, we would reuse the allocation. Now, we allocate a new tight frame. The CL that made this change is https://webrtc-codereview.appspot.com/42469004/. This exposed an uninitialized memory problem in a I420VideoFrame unittest. This CL fixes that unittest.

R=stefan@webrtc.org

Review URL: https://webrtc-codereview.appspot.com/42619004

Cr-Commit-Position: refs/heads/master@{#8593}
git-svn-id: http://webrtc.googlecode.com/svn/trunk@8593 4adac7df-926f-26a2-2b94-8c16560cd09d

webrtc/common_video/i420_video_frame_unittest.cc

diff --git a/webrtc/common_video/i420_video_frame_unittest.cc b/webrtc/common_video/i420_video_frame_unittest.cc
index 45e9016..9c0d700 100644
--- a/webrtc/common_video/i420_video_frame_unittest.cc
+++ b/webrtc/common_video/i420_video_frame_unittest.cc
@@ -108,6 +108,9 @@
   // Frame of larger dimensions.
   EXPECT_EQ(0, small_frame.CreateEmptyFrame(width, height,
                                             stride_y, stride_u, stride_v));
+  memset(small_frame.buffer(kYPlane), 1, small_frame.allocated_size(kYPlane));
+  memset(small_frame.buffer(kUPlane), 2, small_frame.allocated_size(kUPlane));
+  memset(small_frame.buffer(kVPlane), 3, small_frame.allocated_size(kVPlane));
   EXPECT_EQ(0, big_frame.CopyFrame(small_frame));
   EXPECT_TRUE(EqualFrames(small_frame, big_frame));
 }