Added video capture capabilities retrieval and caching to VideoCaptureManager

media/video/capture/fake_video_capture_device.h

Index: media/video/capture/fake_video_capture_device.h
diff --git a/media/video/capture/fake_video_capture_device.h b/media/video/capture/fake_video_capture_device.h
index 174ba06ae7e729161be19d831d1fdec668c94c0b..c316c4268416e1591fe0d961a397568a78b095ae 100644
--- a/media/video/capture/fake_video_capture_device.h
+++ b/media/video/capture/fake_video_capture_device.h
@@ -23,6 +23,8 @@ class MEDIA_EXPORT FakeVideoCaptureDevice : public VideoCaptureDevice {
   // Used for testing. This will make sure the next call to Create will
   // return NULL;
   static void SetFailNextCreate();
+  static void SetNumberOfFakeDevices(int num);
+  static int NumberOfFakeDevices();
 
   static void GetDeviceNames(Names* device_names);
   static void GetDeviceSupportedFormats(const Name& device,
@@ -63,6 +65,7 @@ class MEDIA_EXPORT FakeVideoCaptureDevice : public VideoCaptureDevice {
   std::vector<VideoCaptureCapability> capabilities_roster_;
   int capabilities_roster_index_;
 
+  static int numberOfFakeDevices;

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

I am confused here, how this numberOfFakeDevices is used? and why it is static?

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

Originally there was a file-scope enum =2 that was used during
static GetDeviceNames() to create as many fake video capture
devices. So, there were 2 video capture devices. 

I made this static so we can manipulate during the unittests
how many (fake) video capture devices are in the system, and
add a test case that adds-removes devices and checks that the
DeviceInfo cache introduced works as intended.

Needs to be static because is used from static methods and
only pertains to all Fake VCDs as a community.

[no longer working on chromium, 2013/11/13 16:20:21]

I see, thanks for the clarification.

   static bool fail_next_create_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeVideoCaptureDevice);