Implement GetDefaultOutputDeviceID, GetAssociatedOutputDeviceID and ...

media/audio/audio_manager_unittest.cc

 // Copyright 2013 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 "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "media/audio/audio_manager.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 #if defined(USE_PULSEAUDIO)
@@ skipping 16 matching lines @@
   for (AudioDeviceNames::iterator it = device_names.begin();
        it != device_names.end();
        ++it) {
     EXPECT_FALSE(it->unique_id.empty());
     EXPECT_FALSE(it->device_name.empty());
     VLOG(2) << "Device ID(" << it->unique_id << "), label: " << it->device_name;
   }
 #endif  // defined(USE_PULSEAUDIO)
 }
 
+TEST(AudioManagerTest, GetDefaultOutputStreamParameters) {
+#if defined(OS_WIN) || defined(OS_MACOSX)

[henrika (OOO until Aug 14), 2013/09/07 08:04:25]

Why inside and not outside the complete method?

[tommi (sloooow) - chröme, 2013/09/08 19:11:19]

I'm following the style of the other test in this file.

+  scoped_ptr<AudioManager> audio_manager(AudioManager::Create());
+  ASSERT_TRUE(audio_manager);
+  if (!audio_manager->HasAudioOutputDevices())
+    return;
+
+  AudioParameters params = audio_manager->GetDefaultOutputStreamParameters();
+  EXPECT_TRUE(params.IsValid());
+#endif  // defined(OS_WIN) || defined(OS_MACOSX)
+}
+
+TEST(AudioManagerTest, GetAssociatedOutputDeviceID) {
+#if defined(OS_WIN) || defined(OS_MACOSX)
+  scoped_ptr<AudioManager> audio_manager(AudioManager::Create());
+  ASSERT_TRUE(audio_manager);
+  if (!audio_manager->HasAudioOutputDevices() ||
+      !audio_manager->HasAudioInputDevices()) {
+    return;
+  }
+
+  AudioDeviceNames device_names;
+  audio_manager->GetAudioInputDeviceNames(&device_names);
+  bool found_an_associated_device = false;
+  for (AudioDeviceNames::iterator it = device_names.begin();
+       it != device_names.end();
+       ++it) {
+    EXPECT_FALSE(it->unique_id.empty());
+    EXPECT_FALSE(it->device_name.empty());
+    std::string output_device_id(
+        audio_manager->GetAssociatedOutputDeviceID(it->unique_id));
+    if (!output_device_id.empty()) {
+      VLOG(2) << it->unique_id << " matches with " << output_device_id;
+      found_an_associated_device = true;
+    }
+  }
+
+  EXPECT_TRUE(found_an_associated_device);

[henrika (OOO until Aug 14), 2013/09/07 08:04:25]

Are we sure that this will be true on all boths with hardware?

[tommi (sloooow) - chröme, 2013/09/08 19:11:19]

I think it's a reasonable thing to expect (we have checks for both output and
input devies at the top, if none of them match, it'll be a fairly unusual
setup).  If it turns out to be a problem, it's easy to fix.

+#endif  // defined(OS_WIN) || defined(OS_MACOSX)
+}
 }  // namespace media