Use the device listener in Media on windows

content/browser/system_message_window_win.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/system_message_window_win.h"
 
 #include <dbt.h>
 #include <ks.h>
 #include <ksmedia.h>
 
 #include "base/logging.h"
 #include "base/system_monitor/system_monitor.h"
+#include "base/win/windows_version.h"
 #include "base/win/wrapped_window_proc.h"
 
 namespace content {
 
 namespace {
+
 const wchar_t kWindowClassName[] = L"Chrome_SystemMessageWindow";
 
 // A static map from a device category guid to base::SystemMonitor::DeviceType.
 struct {
   const GUID device_category;
   const base::SystemMonitor::DeviceType device_type;
 } const kDeviceCategoryMap[] = {
   { KSCATEGORY_AUDIO, base::SystemMonitor::DEVTYPE_AUDIO_CAPTURE },
   { KSCATEGORY_VIDEO, base::SystemMonitor::DEVTYPE_VIDEO_CAPTURE },
 };
+
+bool IsCoreAudioSupported() {
+  return (base::win::GetVersion() >= base::win::VERSION_VISTA);
+}
+
 }  // namespace
 
 // Manages the device notification handles for SystemMessageWindowWin.
 class SystemMessageWindowWin::DeviceNotifications {
  public:
   explicit DeviceNotifications(HWND hwnd) : notifications_() {
     Register(hwnd);
   }
 
   ~DeviceNotifications() {
     Unregister();
   }
 
   void Register(HWND hwnd) {
     // Request to receive device notifications.  All applications receive basic
     // notifications via WM_DEVICECHANGE but in order to receive detailed device
     // arrival and removal messages, we need to register.
     DEV_BROADCAST_DEVICEINTERFACE filter = {0};
     filter.dbcc_size = sizeof(filter);
     filter.dbcc_devicetype = DBT_DEVTYP_DEVICEINTERFACE;
     for (int i = 0; i < arraysize(kDeviceCategoryMap); ++i) {
+      // From Vista, we use a device listener in AudioDeviceListenerWin.
+      if (IsCoreAudioSupported() &&

[DaleCurtis, 2012/12/11 19:27:59]

This should use Henrik's CoreAudioUtil class.

[no longer working on chromium, 2012/12/11 20:15:49]

I tried include the media/audio/win/core_audio_util_win.h, but it introduced a
compiling warning something about "no object", not sure if it is because the
content/browser has a stricter compiling flags.

I will double check this with Henrik and Tommi tomorrow.

[no longer working on chromium, 2012/12/12 10:17:16]

including the CoreAudioUtil.h gives me this compiling error:
[1/15] CXX obj\content\browser\content.system_message_window_win.obj
FAILED: ninja -t msvc -r . -o
obj\content\browser\content.system_message_window_win.obj -e environment.x86 --
cl.exe /no
logo /showIncludes /FC
@obj\content\browser\content.system_message_window_win.obj.rsp  /c
..\..\content\browser\system_m
essage_window_win.cc
/Foobj\content\browser\content.system_message_window_win.obj  /Fdcontent.pdb
c:\users\xians\dev\chrome1\src\content\browser\system_message_window_win.cc(31)
: error C2039: 'GetVersion' : is not a m
ember of 'base::win'
c:\users\xians\dev\chrome1\src\content\browser\system_message_window_win.cc(31)
: error C2039: 'VERSION_VISTA' : is not
a member of 'base::win'
c:\users\xians\dev\chrome1\src\content\browser\system_message_window_win.cc(31)
: error C2065: 'VERSION_VISTA' : undecla
red identifier
ninja: build stopped: subcommand failed.

Tommi, do you know why? Does it worth the effort to fix it?

[tommi (sloooow) - chröme, 2012/12/12 13:01:09]

This doesn't look like related to CoreAudioUtil.h but rather that you removed
the #include for base/win/windows_version.h and did not remove your
IsCoreAudioSupported method.

+          KSCATEGORY_AUDIO == kDeviceCategoryMap[i].device_category)
+        continue;
+
       filter.dbcc_classguid = kDeviceCategoryMap[i].device_category;
       DCHECK_EQ(notifications_[i], static_cast<HDEVNOTIFY>(NULL));
       notifications_[i] = RegisterDeviceNotification(
           hwnd, &filter, DEVICE_NOTIFY_WINDOW_HANDLE);
       DPLOG_IF(ERROR, !notifications_[i])
           << "RegisterDeviceNotification failed";
     }
   }
 
   void Unregister() {
@@ skipping 87 matching lines @@
     case WM_DEVICECHANGE:
       return OnDeviceChange(static_cast<UINT>(wparam), lparam);
     default:
       break;
   }
 
   return ::DefWindowProc(hwnd, message, wparam, lparam);
 }
 
 }  // namespace content