Added video capture capabilities retrieval and caching to VideoCaptureManager

content/browser/renderer_host/media/video_capture_controller.h

Index: content/browser/renderer_host/media/video_capture_controller.h
diff --git a/content/browser/renderer_host/media/video_capture_controller.h b/content/browser/renderer_host/media/video_capture_controller.h
index 0e9fd788923341b3d8788a4d047fb40a80887c73..95a45f484dedfc7364325e01554bceac6e809b68 100644
--- a/content/browser/renderer_host/media/video_capture_controller.h
+++ b/content/browser/renderer_host/media/video_capture_controller.h
@@ -104,6 +104,10 @@ class CONTENT_EXPORT VideoCaptureController {
                     VideoCaptureControllerEventHandler* event_handler,
                     int buffer_id);
 
+  const media::VideoCaptureParams& GetVideoCaptureParams() const {

[no longer working on chromium, 2013/11/12 16:48:00]

You should either change the name to video_capture_params() const and make it
inline. Or move the implementation to cc file.

Also, it looks like video_capture_params_ should be accessed only on IO thread,
add a thread check DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));

So I would say that moving the implementation to cc file will be the best.

[mcasas, 2013/11/12 18:10:16]

Done.

+    return video_capture_params_;
+  }
+
  private:
   class VideoCaptureDeviceClient;
 
@@ -140,6 +144,8 @@ class CONTENT_EXPORT VideoCaptureController {
   // state which stops the flow of data to clients.
   VideoCaptureState state_;
 
+  media::VideoCaptureParams video_capture_params_;
+
   base::WeakPtrFactory<VideoCaptureController> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoCaptureController);