Relanding 1143663007: VideoFrame: Separate Pixel Format from Storage Type.

media/base/mac/video_frame_mac_unittests.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/mac/video_frame_mac.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 43 matching lines @@
     EXPECT_EQ(plane_size.width(),
               static_cast<int>(CVPixelBufferGetWidthOfPlane(pb, i)));
     EXPECT_EQ(plane_size.height(),
               static_cast<int>(CVPixelBufferGetHeightOfPlane(pb, i)));
     EXPECT_EQ(frame->data(i), CVPixelBufferGetBaseAddressOfPlane(pb, i));
   }
   CVPixelBufferUnlockBaseAddress(pb, 0);
 }
 
 TEST(VideoFrameMac, CheckFormats) {
+  // CreateFrame() does not support non planar YUV, e.g. NV12.
   const FormatPair format_pairs[] = {
       {VideoFrame::I420, kCVPixelFormatType_420YpCbCr8Planar},
-
       {VideoFrame::YV12, 0},
       {VideoFrame::YV16, 0},
       {VideoFrame::YV12A, 0},
       {VideoFrame::YV24, 0},
   };
 
   gfx::Size size(kWidth, kHeight);
   for (const auto& format_pair : format_pairs) {
     auto frame = VideoFrame::CreateFrame(format_pair.chrome, size,
                                          gfx::Rect(size), size, kTimestamp);
@@ skipping 47 matching lines @@
     ASSERT_TRUE(frame.get());
     EXPECT_EQ(pb.get(), frame->cv_pixel_buffer());
     EXPECT_EQ(format_pair.chrome, frame->format());
 
     frame = nullptr;
     EXPECT_EQ(1, CFGetRetainCount(pb.get()));
   }
 }
 
 }  // namespace media