Win Video Capture: add DirectShow WDM devices capabilities enumeration.

media/video/capture/win/video_capture_device_factory_win.cc

 // Copyright 2014 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 "media/video/capture/win/video_capture_device_factory_win.h"
 
 #include <mfapi.h>
 #include <mferror.h>
 
 #include "base/command_line.h"
@@ skipping 204 matching lines @@
     return;
 
   ScopedComPtr<IEnumMoniker> enum_moniker;
   hr = dev_enum->CreateClassEnumerator(CLSID_VideoInputDeviceCategory,
                                        enum_moniker.Receive(), 0);
   // CreateClassEnumerator returns S_FALSE on some Windows OS when no camera
   // exists. Therefore the FAILED macro can't be used.
   if (hr != S_OK)
     return;
 
-  // Walk the capture devices. No need to check for device presence again, that
-  // is caught in GetDeviceFilter(). "google camera adapter" and old VFW devices
-  // are already skipped in the previous GetDeviceNames() enumeration.
+  // Walk the capture devices. No need to check for device presence again since
+  // that is anyway needed in GetDeviceFilter(). "google camera adapter" and old
+  // VFW devices are already skipped previously in GetDeviceNames() enumeration.
   base::win::ScopedComPtr<IBaseFilter> capture_filter;
-  hr = VideoCaptureDeviceWin::GetDeviceFilter(device,
-                                              capture_filter.Receive());
+  if (device.capture_api_type() == Name::DIRECT_SHOW_WDM) {
+    // Find the "normal" DirectShow device corresponding to the WDM device by
+    // comparing USB IDs.
+    // TODO(mcasas): USB ID comparison won't work with PCIe capture cards.
+    Names device_names;
+    GetDeviceNamesDirectShow(&device_names);
+    const std::string current_device_model = device.GetModel();
+    for (Names::const_iterator iter = device_names.begin();
+        iter != device_names.end(); ++iter) {
+      if ((iter->GetModel().compare(current_device_model) == 0) &&
+          (iter->capture_api_type() == Name::DIRECT_SHOW)) {
+        DVLOG(1) << "Enumerating formats for a WDM device.";
+        hr = VideoCaptureDeviceWin::GetDeviceFilter(*iter,

[perkj_chrome, 2014/09/01 09:57:43]

GetDeviceFilter need the correct Name::id. So why is this necessary? Why not
make sure that |device| have the correct id from the beginning when the name is
enumerated? 

+                                                    capture_filter.Receive());
+        break;
+      }
+    }
+  } else {
+    hr = VideoCaptureDeviceWin::GetDeviceFilter(device,
+                                                capture_filter.Receive());
+  }
   if (!capture_filter) {
     DVLOG(2) << "Failed to create capture filter.";
     return;
   }
 
   base::win::ScopedComPtr<IPin> output_capture_pin(
       VideoCaptureDeviceWin::GetPin(capture_filter,
                                     PINDIR_OUTPUT,
                                     PIN_CATEGORY_CAPTURE));
   if (!output_capture_pin) {
@@ skipping 177 matching lines @@
     const Name& device,
     VideoCaptureFormats* formats) {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (use_media_foundation_)
     GetDeviceSupportedFormatsMediaFoundation(device, formats);
   else
     GetDeviceSupportedFormatsDirectShow(device, formats);
 }
 
 }  // namespace media