Win Video Capture: Add device blacklisting, used for IPCamera and YouCam, collect UMA for those

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"
 #include "base/lazy_instance.h"
+#include "base/metrics/histogram.h"
 #include "base/strings/string_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/win/metro.h"
 #include "base/win/scoped_co_mem.h"
 #include "base/win/scoped_variant.h"
 #include "base/win/windows_version.h"
 #include "media/base/media_switches.h"
 #include "media/video/capture/win/video_capture_device_mf_win.h"
 #include "media/video/capture/win/video_capture_device_win.h"
 
 using base::win::ScopedCoMem;
 using base::win::ScopedComPtr;
 using base::win::ScopedVariant;
 using Name = media::VideoCaptureDevice::Name;
 using Names = media::VideoCaptureDevice::Names;
 
 namespace media {
 
+// Some devices cause crashes when trying to be enumerated. This enum is defined
+// for the purposes of UMA collection. Existing entries cannot be removed.
+enum BlacklistedCameraNames{
+  BLACKLISTED_CAMERA_IP_CAMERA,
+  BLACKLISTED_CAMERA_CYBERLINK_WEBCAM_SPLITTER,
+  BLACKLISTED_CAMERA_MAX  // This one must be last.
+};
+
 // Lazy Instance to initialize the MediaFoundation Library.
 class MFInitializerSingleton {
  public:
   MFInitializerSingleton() { MFStartup(MF_VERSION, MFSTARTUP_LITE); }
   ~MFInitializerSingleton() { MFShutdown(); }
 };
 
 static base::LazyInstance<MFInitializerSingleton> g_mf_initialize =
     LAZY_INSTANCE_INITIALIZER;
 
+// Blacklisted devices are identified by a characteristic substring of the name.
+static const char* kBlacklistedCameraNames[] = {
+  {"IP Camera [JPEG/MJPEG]" },

[tommi (sloooow) - chröme, 2014/10/12 09:46:03]

nit: no space before } (for consistency)

[mcasas, 2014/10/12 14:45:39]

Done.

+  {"CyberLink Webcam Splitter"}

[tommi (sloooow) - chröme, 2014/10/12 09:46:03]

Can you point me to a report where we encountered this one?  Do you know which
progid or clsid this corresponds to?

[mcasas, 2014/10/12 14:45:39]

Sure, there are a few in the other bug linked to this CL:
hhtp://crbug.com/421991
I'm afraid I don't ... :(

+};
+
 static void EnsureMediaFoundationInit() {
   g_mf_initialize.Get();
 }
 
 static bool LoadMediaFoundationDlls() {
   static const wchar_t* const kMfDLLs[] = {
     L"%WINDIR%\\system32\\mf.dll",
     L"%WINDIR%\\system32\\mfplat.dll",
     L"%WINDIR%\\system32\\mfreadwrite.dll",
   };
@@ skipping 33 matching lines @@
 
 static bool EnumerateVideoDevicesMediaFoundation(IMFActivate*** devices,
                                                  UINT32* count) {
   ScopedComPtr<IMFAttributes> attributes;
   if (!PrepareVideoCaptureAttributesMediaFoundation(attributes.Receive(), 1))
     return false;
 
   return SUCCEEDED(MFEnumDeviceSources(attributes, devices, count));
 }
 
+static bool IsDeviceBlackListed(const std::string& name) {
+  DCHECK_EQ(static_cast<int>(arraysize(kBlacklistedCameraNames)),
+            BLACKLISTED_CAMERA_MAX);
+  for (size_t i = 0; i < arraysize(kBlacklistedCameraNames); ++i) {
+    if (name.find (kBlacklistedCameraNames[i]) != std::string::npos) {

[tommi (sloooow) - chröme, 2014/10/12 09:46:03]

no space before (

What about StartsWith? (string_util.h)

[mcasas, 2014/10/12 14:45:39]

Done.

+      DVLOG(1) << "Enumerated blacklisted device: " << name;
+      UMA_HISTOGRAM_ENUMERATION("Media.VideoCapture.BlacklistedDevice",
+          i, BLACKLISTED_CAMERA_MAX + 1);
+      return true;
+    }
+  }
+  return false;
+}
+
 static void GetDeviceNamesDirectShow(
     const CLSID& class_id,
     const Name::CaptureApiType capture_api_type,
     Names* device_names) {
   DCHECK(device_names);
   DVLOG(1) << " GetDeviceNamesDirectShow";
 
   ScopedComPtr<ICreateDevEnum> dev_enum;
   HRESULT hr = dev_enum.CreateInstance(CLSID_SystemDeviceEnum, NULL,
                                        CLSCTX_INPROC);
@@ skipping 24 matching lines @@
 
     if (FAILED(hr) || name.type() != VT_BSTR)
       continue;
 
     // Ignore all VFW drivers and the special Google Camera Adapter.
     // Google Camera Adapter is not a real DirectShow camera device.
     // VFW are very old Video for Windows drivers that can not be used.
     const wchar_t* str_ptr = V_BSTR(&name);
     // Name of a fake DirectShow filter that exist on computers with
     // GTalk installed.
     static const char kGoogleCameraAdapter[] = "google camera adapter";

[tommi (sloooow) - chröme, 2014/10/12 09:46:03]

What about moving this into the blacklist now that we have one?
We'd have to be careful to make sure it's not case sensitive.

For now, if you could add a TODO here and ping perkj about the details, that
would be good.

[mcasas, 2014/10/12 14:45:39]

Done, moved "google camera adapter" to the blacklist, and
updated the comments, enum and histogram. I took your 
comment of using StartsWith(), which is used case-insensitively.

     if (wcsstr(str_ptr, L"(VFW)") != NULL ||
         LowerCaseEqualsASCII(str_ptr,
                              str_ptr + arraysize(kGoogleCameraAdapter) - 1,
                              kGoogleCameraAdapter)) {
       continue;
     }
 
     const std::string device_name(base::SysWideToUTF8(str_ptr));
+    if (IsDeviceBlackListed(device_name))
+      continue;
     name.Reset();
     hr = prop_bag->Read(L"DevicePath", name.Receive(), 0);
     std::string id;
     if (FAILED(hr) || name.type() != VT_BSTR) {
       id = device_name;
     } else {
       DCHECK_EQ(name.type(), VT_BSTR);
       id = base::SysWideToUTF8(V_BSTR(&name));
     }
     device_names->push_back(Name(device_name, id, capture_api_type));
@@ skipping 271 matching lines @@
     const Name& device,
     VideoCaptureFormats* formats) {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (use_media_foundation_)
     GetDeviceSupportedFormatsMediaFoundation(device, formats);
   else
     GetDeviceSupportedFormatsDirectShow(device, formats);
 }
 
 }  // namespace media