Added video capture capabilities retrieval and caching to VideoCaptureManager

content/browser/renderer_host/media/media_stream_manager.cc

Index: content/browser/renderer_host/media/media_stream_manager.cc
diff --git a/content/browser/renderer_host/media/media_stream_manager.cc b/content/browser/renderer_host/media/media_stream_manager.cc
index 2fe5e6fd2af6dc5f5a97f9c0da932f6c0c2177be..c26b90f8253f05d31117c642cc7f2e3010c245c5 100644
--- a/content/browser/renderer_host/media/media_stream_manager.cc
+++ b/content/browser/renderer_host/media/media_stream_manager.cc
@@ -889,6 +889,20 @@ void MediaStreamManager::DevicesEnumerated(
   DCHECK_GE(active_enumeration_ref_count_[stream_type], 0);
 }
 
+void MediaStreamManager::DeviceCapabilitiesEnumerated(
+    const StreamDeviceInfo& device_info,
+    const media::VideoCaptureCapabilities& capabilities) {
+

[tommi (sloooow) - chröme, 2013/10/25 14:15:34]

remove this empty line.

If there are plans for the implementation here, it might be good to document
them with a todo.

[mcasas, 2013/10/28 12:50:08]

Done & done.

+  media::VideoCaptureCapabilities::const_iterator format;
+  for (format = capabilities.begin(); format != capabilities.end(); ++format) {
+    DCHECK_GE(format->width, 1);
+    DCHECK_GE(format->height, 1);
+    DCHECK_GE(format->frame_rate, 1);
+    DVLOG(1) << " Device format (" << format->width << "x" << format->height
+             << ")@" << format->frame_rate << "fps";
+  }
+}
+
 void MediaStreamManager::Error(MediaStreamType stream_type,
                                int capture_session_id,
                                MediaStreamProviderError error) {