Added video capture capabilities retrieval and caching to VideoCaptureManager

content/browser/renderer_host/media/video_capture_controller.cc

Index: content/browser/renderer_host/media/video_capture_controller.cc
diff --git a/content/browser/renderer_host/media/video_capture_controller.cc b/content/browser/renderer_host/media/video_capture_controller.cc
index 9778a9c491260df5054420e182fd6d347491a1ce..5cd4b3d9a7147b6025be84dfe846bde053b05e78 100644
--- a/content/browser/renderer_host/media/video_capture_controller.cc
+++ b/content/browser/renderer_host/media/video_capture_controller.cc
@@ -153,6 +153,8 @@ void VideoCaptureController::AddClient(
            << ", " << params.session_id
            << ")";
 
+  video_capture_params_ = params;

[ncarter (slow), 2013/11/12 20:56:46]

It may make sense to update this only if it's the first client (params is a
request, and the policy implemented in VCM is that the first requesting client
gets its way). You could do this either by checking that the format in
video_capture_params_ is valid, or else by checking whether controller_clients_
is empty.

[mcasas, 2013/11/13 11:40:37]

Done.

+
   // Signal error in case device is already in error state.
   if (state_ == VIDEO_CAPTURE_STATE_ERROR) {
     event_handler->OnError(id);
@@ -229,6 +231,12 @@ void VideoCaptureController::ReturnBuffer(
   buffer_pool_->RelinquishConsumerHold(buffer_id, 1);
 }
 
+const media::VideoCaptureParams&
+VideoCaptureController::GetVideoCaptureParams() const {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  return video_capture_params_;
+}
+
 scoped_refptr<media::VideoFrame>
 VideoCaptureController::VideoCaptureDeviceClient::ReserveOutputBuffer(
     const gfx::Size& size) {