Added video capture capabilities retrieval and caching to VideoCaptureManager

content/browser/renderer_host/media/media_stream_manager.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.
 
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 
 #include <list>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 871 matching lines @@
         // Post the request to UI for permission approval.
         PostRequestToUI(*it);
         break;
     }
   }
   label_list.clear();
   --active_enumeration_ref_count_[stream_type];
   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) {
   // Find the device for the error call.
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   DVLOG(1) << "Error("
            << "{stream_type = " << stream_type << "} ,"
            << "{capture_session_id = " << capture_session_id <<  "})";
 
 
@@ skipping 293 matching lines @@
   }
 
   // Always do enumeration even though some enumeration is in progress,
   // because those enumeration commands could be sent before these devices
   // change.
   ++active_enumeration_ref_count_[stream_type];
   GetDeviceManager(stream_type)->EnumerateDevices(stream_type);
 }
 
 }  // namespace content