Handle audio device changes on Mac (Take 2).

media/audio/mac/audio_manager_mac.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 <CoreAudio/AudioHardware.h>
 
 #include <string>
 
+#include "base/bind.h"
 #include "base/command_line.h"
 #include "base/mac/mac_logging.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/sys_string_conversions.h"
 #include "media/audio/mac/audio_input_mac.h"
 #include "media/audio/mac/audio_low_latency_input_mac.h"
 #include "media/audio/mac/audio_low_latency_output_mac.h"
 #include "media/audio/mac/audio_manager_mac.h"
 #include "media/audio/mac/audio_output_mac.h"
 #include "media/audio/mac/audio_synchronized_mac.h"
 #include "media/audio/mac/audio_unified_mac.h"
+#include "media/base/bind_to_loop.h"
 #include "media/base/limits.h"
 #include "media/base/media_switches.h"
 
 namespace media {
 
 // Maximum number of output streams that can be open simultaneously.
 static const int kMaxOutputStreams = 50;
 
 static bool HasAudioHardware(AudioObjectPropertySelector selector) {
   AudioDeviceID output_device_id = kAudioObjectUnknown;
@@ skipping 193 matching lines @@
   }
 
   if (result) {
     OSSTATUS_DLOG(WARNING, result) << "Unable to query device " << device_id
                                    << " for AudioDeviceID";
   }
 
   return audio_device_id;
 }
 
+// Property address to monitor for device changes.

[scherkus (not reviewing), 2012/11/01 01:20:14]

how about replacing this comment by changing kPA to kDeviceChangePropertyAddress
(you would do this anyway if you had multiple property addresses)

[DaleCurtis, 2012/11/01 01:28:17]

Whoops, yeah that old name is a bad choice now. Done.

+static const AudioObjectPropertyAddress kPropertyAddress = {
+  kAudioHardwarePropertyDefaultOutputDevice,
+  kAudioObjectPropertyScopeGlobal,
+  kAudioObjectPropertyElementMaster
+};
+
+// Callback from the system when the default device changes.  This can be called
+// either on the main thread or on an audio thread managed by the system
+// depending on what kAudioHardwarePropertyRunLoop is set to.
+static OSStatus OnDefaultDeviceChangedCallback(
+    AudioObjectID object,
+    UInt32 size,
+    const AudioObjectPropertyAddress addresses[],
+    void* context) {
+  static_cast<base::Closure*>(context)->Run();
+  return noErr;
+}
+
 AudioManagerMac::AudioManagerMac() {
   SetMaxOutputStreamsAllowed(kMaxOutputStreams);
+
+  // Register a callback for device changes.
+  listener_cb_ = BindToLoop(GetMessageLoop(), base::Bind(
+      &AudioManagerMac::NotifyAllOutputDeviceChangeListeners,
+      base::Unretained(this)));
+
+  OSStatus result = AudioObjectAddPropertyListener(
+      kAudioObjectSystemObject,
+      &kPropertyAddress,
+      &OnDefaultDeviceChangedCallback,
+      &listener_cb_);
+
+  // Clear |listener_cb_| so we don't try to remove the listener later.

[scherkus (not reviewing), 2012/11/01 01:20:14]

comment isn't adding value given proximity to dtor

[DaleCurtis, 2012/11/01 01:28:17]

Done.

+  if (result != noErr) {
+    OSSTATUS_DLOG(ERROR, result) << "AudioObjectAddPropertyListener() failed!";
+    listener_cb_.Reset();
+  }
 }
 
 AudioManagerMac::~AudioManagerMac() {
+  // Tear down our listener.

[scherkus (not reviewing), 2012/11/01 01:20:14]

comment isn't adding value

[DaleCurtis, 2012/11/01 01:28:17]

Done.

+  if (!listener_cb_.is_null()) {
+    OSStatus result = AudioObjectRemovePropertyListener(
+        kAudioObjectSystemObject,
+        &kPropertyAddress,
+        &OnDefaultDeviceChangedCallback,
+        &listener_cb_);
+    OSSTATUS_DLOG_IF(ERROR, result != noErr, result)
+        << "AudioObjectRemovePropertyListener() failed!";
+  }
+
   Shutdown();
 }
 
 bool AudioManagerMac::HasAudioOutputDevices() {
   return HasAudioHardware(kAudioHardwarePropertyDefaultOutputDevice);
 }
 
 bool AudioManagerMac::HasAudioInputDevices() {
   return HasAudioHardware(kAudioHardwarePropertyDefaultInputDevice);
 }
@@ skipping 55 matching lines @@
     stream = new AUAudioInputStream(this, params, audio_device_id);
 
   return stream;
 }
 
 AudioManager* CreateAudioManager() {
   return new AudioManagerMac();
 }
 
 }  // namespace media