Chromium Code Reviews| Index: media/video/gpu_memory_buffer_video_frame_pool_unittest.cc |
| diff --git a/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc b/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc |
| index f0778859147d84649569ff35142dcd1dfc59f73f..e1db98036d4d53e6a0db0639cd65935b9da5608d 100644 |
| --- a/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc |
| +++ b/media/video/gpu_memory_buffer_video_frame_pool_unittest.cc |
| @@ -72,18 +72,21 @@ class GpuMemoryBufferVideoFramePoolTest : public ::testing::Test { |
| DCHECK_LE(dimension, kDimension); |
| gfx::Size size(dimension, dimension); |
| - return media::VideoFrame::WrapExternalYuvData( |
| - media::PIXEL_FORMAT_YV12, // format |
| - size, // coded_size |
| - gfx::Rect(size), // visible_rect |
| - size, // natural_size |
| - size.width(), // y_stride |
| - size.width() / 2, // u_stride |
| - size.width() / 2, // v_stride |
| - y_data, // y_data |
| - u_data, // u_data |
| - v_data, // v_data |
| - base::TimeDelta()); // timestamp |
| + scoped_refptr<VideoFrame> video_frame = |
| + media::VideoFrame::WrapExternalYuvData( |
| + media::PIXEL_FORMAT_YV12, // format |
| + size, // coded_size |
| + gfx::Rect(size), // visible_rect |
| + size, // natural_size |
| + size.width(), // y_stride |
| + size.width() / 2, // u_stride |
| + size.width() / 2, // v_stride |
| + y_data, // y_data |
| + u_data, // u_data |
| + v_data, // v_data |
| + base::TimeDelta()); // timestamp |
| + EXPECT_NE(video_frame.get(), nullptr); |
|
mcasas
2016/01/12 17:46:16
ASSERT_TRUE(video_frame);
emircan
2016/01/14 23:07:40
ASSERT has the same problem as above.
|
| + return video_frame; |
| } |
| protected: |