Video Capture: extract storage info from pixel format in VideoCaptureFormat.

media/video/capture/video_capture_device.h

Index: media/video/capture/video_capture_device.h
diff --git a/media/video/capture/video_capture_device.h b/media/video/capture/video_capture_device.h
index 6fbfd7d2ef8c8518d35dbc73d3eb50ff680c9875..924fb9f0828de1eeb91402e59823429dcbeeeeae 100644
--- a/media/video/capture/video_capture_device.h
+++ b/media/video/capture/video_capture_device.h
@@ -246,8 +246,7 @@ class MEDIA_EXPORT VideoCaptureDevice {
     // The output buffer stays reserved and mapped for use until the Buffer
     // object is destroyed or returned.
     virtual scoped_ptr<Buffer> ReserveOutputBuffer(
-        media::VideoPixelFormat format,
-        const gfx::Size& dimensions) = 0;
+        const VideoCaptureFormat& frame_format) = 0;

[miu, 2015/06/13 00:41:37]

Hmm...We talked about this in a similar change a few months back, where: 1) the
frame_rate field of VideoCaptureFormat is always unused; and 2) you're burdening
all client code with creating a VideoCaptureFormat struct to make a method call.
 Even with this new, 3rd argument I still feel you should just pass 3 args
instead of the struct.  We can revisit this when it becomes 4+ args.  ;-)

[mcasas, 2015/06/16 23:14:01]

Done.

 
     // Captured new video data, held in |frame| or |buffer|, respectively for
     // OnIncomingCapturedVideoFrame() and  OnIncomingCapturedBuffer().