MediaStreamTrack.getSources list is now updated when input devices is disabled on Windows

media/audio/win/audio_device_listener_win.cc

Index: media/audio/win/audio_device_listener_win.cc
diff --git a/media/audio/win/audio_device_listener_win.cc b/media/audio/win/audio_device_listener_win.cc
index d7e017cbd0311272a53bfb3c4480d9e27aabff52..b2727e6797b33ece6be847ff42c6b080a9476835 100644
--- a/media/audio/win/audio_device_listener_win.cc
+++ b/media/audio/win/audio_device_listener_win.cc
@@ -125,8 +125,19 @@ STDMETHODIMP AudioDeviceListenerWin::OnDeviceRemoved(LPCWSTR device_id) {
 
 STDMETHODIMP AudioDeviceListenerWin::OnDeviceStateChanged(LPCWSTR device_id,
                                                           DWORD new_state) {
-  if (new_state != DEVICE_STATE_ACTIVE && new_state != DEVICE_STATE_NOTPRESENT)
-    return S_OK;
+  std::string new_state_str("UNDEFINED");

[xians, 2014/08/12 11:59:09]

Have you verified that you only get one event when you plugin or unplug a
device?

[henrika (OOO until Aug 14), 2014/08/12 12:04:39]

I do get two callbacks but with the same new_state, hence the old filter did not
make a difference.

+  if (new_state == DEVICE_STATE_ACTIVE)
+    new_state_str = "DEVICE_STATE_ACTIVE";
+  else if (new_state == DEVICE_STATE_DISABLED)
+    new_state_str = "DEVICE_STATE_DISABLED";
+  else if (new_state == DEVICE_STATE_NOTPRESENT)
+    new_state_str = "DEVICE_STATE_NOTPRESENT";
+  else if (new_state == DEVICE_STATE_UNPLUGGED)
+    new_state_str = "DEVICE_STATE_UNPLUGGED";
+
+  DVLOG(1) << "AudioDeviceListenerWin::OnDeviceStateChanged () "

[xians, 2014/08/12 11:59:09]

I am not sure if this logging adds any value after you verify the behaviours.

[henrika (OOO until Aug 14), 2014/08/12 12:04:39]

It would have done this debugging more easy if it had existed... I find it
useful to track down corner cases.

+           << "device_id: " << device_id
+           << ", new_state: " << new_state_str;
 
   base::SystemMonitor* monitor = base::SystemMonitor::Get();
   if (monitor)