Added video capture capabilities retrieval and caching to VideoCaptureManager

media/video/capture/video_capture_device.h

Index: media/video/capture/video_capture_device.h
diff --git a/media/video/capture/video_capture_device.h b/media/video/capture/video_capture_device.h
index 2e60256a8f121f232b950e750391ac640b9e8484..40692f7692f5632d0109d90e84f82c7e1fa70508 100644
--- a/media/video/capture/video_capture_device.h
+++ b/media/video/capture/video_capture_device.h
@@ -18,6 +18,7 @@
 #include "base/logging.h"
 #include "base/time/time.h"
 #include "media/base/media_export.h"
+#include "media/base/video_frame.h"
 #include "media/video/capture/video_capture_types.h"

[perkj_chrome, 2013/11/06 17:05:51]

Why is this changed in this cl ? 

[mcasas, 2013/11/06 20:10:20]

I knew it will raise eyebrows, so I added a comment, but it was lost in time.
Reason: [1] "do not rely on the symbol being brought in transitively via headers
not directly included." And we use VideoFrame. Since I'm modifying the file, I
clean it up.

[1]
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Forwar...

 
 namespace media {
@@ -113,6 +114,7 @@ class MEDIA_EXPORT VideoCaptureDevice {
       : public NON_EXPORTED_BASE(std::list<Name>) {
    public:
     // Returns NULL if no entry was found by that ID.
+    // TODO(mcasas): Possibly not used anymore, double check.

[perkj_chrome, 2013/11/06 17:05:51]

You can find out by removing it and see if it compiles.

[mcasas, 2013/11/06 20:10:20]

You're a genius! I'll try right away.

It worked! Removing it.

     Name* FindById(const std::string& id);
 
     // Allow generated copy constructor and assignment.