Add PIXEL_FORMAT_NV12 into VideoPixelFormat

content/browser/renderer_host/media/video_capture_controller_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.
 
 // Unit test for VideoCaptureController.
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 614 matching lines @@
   Mock::VerifyAndClearExpectations(client_a_.get());
 
   // Second client connects after the error state. It also should get told of
   // the error.
   EXPECT_CALL(*client_b_, DoError(route_id)).Times(1);
   controller_->AddClient(
       route_id, client_b_.get(), base::kNullProcessHandle, 200, session_200);
   Mock::VerifyAndClearExpectations(client_b_.get());
 }
 
+TEST_F(VideoCaptureControllerTest, CaptureAllVideoFormats) {

[mcasas, 2014/12/04 21:09:06]

Suggest test name like: CaptureInEachVideoFormatInSequence.

[zhaoze.zhou, 2014/12/04 21:54:16]

Done.

+  for (int format = 0; format <= media::PIXEL_FORMAT_MAX; format++) {

[mcasas, 2014/12/04 21:09:07]

for (VideoPixelFormat format = 0; format < PIXEL_FORMAT_UNKNOWN;  ++format) {

[zhaoze.zhou, 2014/12/04 21:54:16]

for this, it will need to cast format to int when doing ++, so can I keep it to
int?

[mcasas, 2014/12/04 23:11:53]

Ah, true. Leave it then.

+    media::VideoCaptureParams params;
+    params.requested_format = media::VideoCaptureFormat(
+        gfx::Size(320, 240), 30, media::VideoPixelFormat(format));
+
+    gfx::Size capture_resolution(320, 240);

[mcasas, 2014/12/04 21:09:06]

const?

[zhaoze.zhou, 2014/12/04 21:54:16]

Done.

+
+    const VideoCaptureControllerID route(0x99);
+
+    // add clients
+    controller_->AddClient(route,
+                           client_a_.get(),
+                           base::kNullProcessHandle,
+                           100,
+                           params);
+    ASSERT_EQ(1, controller_->GetClientCount());
+
+    // Now, simulate an incoming captured buffer from the capture device.
+    scoped_refptr<media::VideoCaptureDevice::Client::Buffer> buffer;
+    buffer =
+        device_->ReserveOutputBuffer(media::VideoFrame::I420,
+                                     capture_resolution);
+    ASSERT_TRUE(buffer.get());
+
+    // Captured a new video frame
+    device_->OnIncomingCapturedData(
+              static_cast<unsigned char*>(buffer.get()->data()),
+              buffer.get()->size(),
+              params.requested_format,
+              0,
+              base::TimeTicks());
+
+    base::RunLoop().RunUntilIdle();
+    Mock::VerifyAndClearExpectations(client_a_.get());
+  }
+}
+
 }  // namespace content