media::VideoFrame::WrapExternalSharedMemory

content/browser/renderer_host/media/web_contents_video_capture_device.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.
 //
 // Implementation notes: This needs to work on a variety of hardware
 // configurations where the speed of the CPU and GPU greatly affect overall
 // performance. Spanning several threads, the process of capturing has been
 // split up into four conceptual stages:
 //
 //   1. Reserve Buffer: Before a frame can be captured, a slot in the consumer's
@@ skipping 630 matching lines @@
       input.config() != SkBitmap::kARGB_8888_Config ||
       input.width() < 2 || input.height() < 2) {
     DVLOG(1) << "input unacceptable (size="
              << input.getSize()
              << ", ready=" << input.readyToDraw()
              << ", config=" << input.config() << ')';
     return;
   }
 
   // Sanity-check the output buffer.
-  if (output->format() != media::VideoFrame::YV12) {
+  if (output->format() != media::VideoFrame::I420) {

[miu, 2013/08/08 18:03:31]

This shouldn't work: A very subtle detail, but the U and V planes are swapped
between YV12 and I420.  ;-)

[sheu, 2013/08/08 18:34:09]

They're swapped, but we access the frame's data through frame->data(PLANE), so
the planar data is being dumped into the right locations

     NOTREACHED();
     return;
   }
 
   // Calculate the width and height of the content region in the |output|, based
   // on the aspect ratio of |input|.
   gfx::Rect region_in_frame = ComputeYV12LetterboxRegion(
       output->coded_size(), gfx::Size(input.width(), input.height()));
 
   // Scale the bitmap to the required size, if necessary.
@@ skipping 607 matching lines @@
 void WebContentsVideoCaptureDevice::DeAllocate() {
   impl_->DeAllocate();
 }
 
 const media::VideoCaptureDevice::Name&
 WebContentsVideoCaptureDevice::device_name() {
   return device_name_;
 }
 
 }  // namespace content