Replace AudioManager::GetAudio*DeviceNames with AudioManager::GetAudio*DeviceDescriptions

content/browser/renderer_host/media/media_devices_manager.cc

Index: content/browser/renderer_host/media/media_devices_manager.cc
diff --git a/content/browser/renderer_host/media/media_devices_manager.cc b/content/browser/renderer_host/media/media_devices_manager.cc
index 8f85f5ced95d64228031c660bd036c89da6156e4..ccc3b4d4e2ea3f5bbf0e3f6a0865f14bf1f71bf8 100644
--- a/content/browser/renderer_host/media/media_devices_manager.cc
+++ b/content/browser/renderer_host/media/media_devices_manager.cc
@@ -55,18 +55,14 @@ MediaDeviceInfoArray EnumerateAudioDevicesOnDeviceThread(
   DCHECK(audio_manager->GetTaskRunner()->BelongsToCurrentThread());
 
   MediaDeviceInfoArray snapshot;
-  media::AudioDeviceNames device_names;
+  media::AudioDeviceDescriptions device_descriptions;
   if (is_input)
-    audio_manager->GetAudioInputDeviceNames(&device_names);
+    audio_manager->GetAudioInputDeviceDescriptions(&device_descriptions);
   else
-    audio_manager->GetAudioOutputDeviceNames(&device_names);
+    audio_manager->GetAudioOutputDeviceDescriptions(&device_descriptions);
 
-  for (const media::AudioDeviceName& name : device_names) {
-    snapshot.emplace_back(
-        name.unique_id, name.device_name,
-        is_input ? audio_manager->GetGroupIDInput(name.unique_id)
-                 : audio_manager->GetGroupIDOutput(name.unique_id));
-  }
+  for (const media::AudioDeviceDescription& description : device_descriptions)

[DaleCurtis, 2016/12/12 22:13:44]

Does snapshot.insert(snapshot.end(), device_descriptions.begin(),
device_descriptions.end()) work?

[o1ka, 2016/12/13 09:33:58]

No, unless we have a conversion from AudioDeviceDescription to MediaDeviceInfo.

[DaleCurtis, 2016/12/13 20:16:48]

I was hoping the insert() operator would be smart enough to figure out the
conversion; but we probably have explicit on the constructor which prevents
this.

+    snapshot.emplace_back(description);
 
   return snapshot;
 }