Refactor audio manager for Android to avoid heavy tasks at startup

media/audio/android/audio_android_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/basictypes.h"
 #include "base/file_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop/message_loop.h"
 #include "base/path_service.h"
 #include "base/strings/stringprintf.h"
@@ skipping 66 matching lines @@
       return std::string("CHANNEL_LAYOUT_UNSUPPORTED");
   }
 }
 
 static double ExpectedTimeBetweenCallbacks(AudioParameters params) {
   return (base::TimeDelta::FromMicroseconds(
               params.frames_per_buffer() * base::Time::kMicrosecondsPerSecond /
               static_cast<double>(params.sample_rate()))).InMillisecondsF();
 }
 
+// Helper method which verifies that the device list starts with a valid
+// default device name followed by non-default device names.
+static void CheckDeviceNames(const AudioDeviceNames& device_names) {
+  VLOG(2) << "Got " << device_names.size() << " audio devices.";
+  if (device_names.empty()) {
+    // Log a warning so we can see the status on the build bots.  No need to
+    // break the test though since this does successfully test the code and
+    // some failure cases.
+    LOG(WARNING) << "No input devices detected";
+  }
+
+  AudioDeviceNames::const_iterator it = device_names.begin();
+
+  // The first device in the list should always be the default device.
+  EXPECT_EQ(std::string(AudioManagerBase::kDefaultDeviceName),
+            it->device_name);
+  EXPECT_EQ(std::string(AudioManagerBase::kDefaultDeviceId), it->unique_id);
+  ++it;
+
+  // Other devices should have non-empty name and id and should not contain
+  // default name or id.
+  while (it != device_names.end()) {
+    EXPECT_FALSE(it->device_name.empty());
+    EXPECT_FALSE(it->unique_id.empty());
+    VLOG(2) << "Device ID(" << it->unique_id
+            << "), label: " << it->device_name;
+    EXPECT_NE(std::string(AudioManagerBase::kDefaultDeviceName),
+              it->device_name);
+    EXPECT_NE(std::string(AudioManagerBase::kDefaultDeviceId),
+              it->unique_id);
+    ++it;
+  }
+}
+
 std::ostream& operator<<(std::ostream& os, const AudioParameters& params) {
   using namespace std;
   os << endl << "format: " << FormatToString(params.format()) << endl
      << "channel layout: " << LayoutToString(params.channel_layout()) << endl
      << "sample rate: " << params.sample_rate() << endl
      << "bits per sample: " << params.bits_per_sample() << endl
      << "frames per buffer: " << params.frames_per_buffer() << endl
      << "channels: " << params.channels() << endl
      << "bytes per buffer: " << params.GetBytesPerBuffer() << endl
      << "bytes per second: " << params.GetBytesPerSecond() << endl
@@ skipping 420 matching lines @@
 
 // Check if low-latency output is supported and log the result as output.
 TEST_F(AudioAndroidTest, IsAudioLowLatencySupported) {
   AudioManagerAndroid* manager =
       static_cast<AudioManagerAndroid*>(audio_manager());
   bool low_latency = manager->IsAudioLowLatencySupported();
   low_latency ? VLOG(0) << "Low latency output is supported"
               : VLOG(0) << "Low latency output is *not* supported";
 }
 
+// Verify input device enumeration.

[tommi (sloooow) - chröme, 2013/12/13 11:30:13]

nit: Since these two tests are now generally useful for all platforms that
support device enumeration, can you add a TODO here to move them to the more
generic unit test file for AudioManager that Joi added?

[henrika (OOO until Aug 14), 2013/12/13 14:15:27]

My bad; I should have mentioned that I took them from the existing
AudioManagerTest. Is that the one you are referring to? I.e., they are in use
today. I just wanted to move it into the unit test for Android audio to get a
broader test in one place.

+TEST_F(AudioAndroidTest, GetAudioInputDeviceNames) {
+  if (!audio_manager()->HasAudioInputDevices())
+    return;
+  AudioDeviceNames devices;
+  audio_manager()->GetAudioInputDeviceNames(&devices);
+  CheckDeviceNames(devices);
+}
+
+// Verify output device enumeration.
+TEST_F(AudioAndroidTest, GetAudioOutputDeviceNames) {
+  if (!audio_manager()->HasAudioOutputDevices())
+    return;
+  AudioDeviceNames devices;
+  audio_manager()->GetAudioOutputDeviceNames(&devices);
+  CheckDeviceNames(devices);
+}
+
 // Ensure that a default input stream can be created and closed.
 TEST_F(AudioAndroidTest, CreateAndCloseInputStream) {
   AudioParameters params = GetDefaultInputStreamParameters();
+  const char kInvalidId1[] = "_InVaLiDdEvIcEId_";
   AudioInputStream* ais = audio_manager()->MakeAudioInputStream(
+      params, kInvalidId1);
+  EXPECT_FALSE(ais);
+  const char kInvalidId2[] = "192837464";
+  ais = audio_manager()->MakeAudioInputStream(
+      params, kInvalidId2);
+  EXPECT_FALSE(ais);
+  ais = audio_manager()->MakeAudioInputStream(
       params, AudioManagerBase::kDefaultDeviceId);
   EXPECT_TRUE(ais);
   ais->Close();
 }
 
 // Ensure that a default output stream can be created and closed.
 // TODO(henrika): should we also verify that this API changes the audio mode
 // to communication mode, and calls RegisterHeadsetReceiver, the first time
 // it is called?
 TEST_F(AudioAndroidTest, CreateAndCloseOutputStream) {
@@ skipping 211 matching lines @@
   // something that has been added by the test.
   EXPECT_TRUE(ais->Open());
   EXPECT_TRUE(aos->Open());
   ais->Start(&full_duplex);
   aos->Start(&full_duplex);
   VLOG(1) << "HINT: an estimate of the extra FIFO delay will be updated "
           << "once per second during this test.";
   VLOG(0) << ">> Speak into the mic and listen to the audio in loopback...";
   fflush(stdout);
   base::PlatformThread::Sleep(base::TimeDelta::FromSeconds(20));
   printf("\n");

[tommi (sloooow) - chröme, 2013/12/13 11:30:13]

do you mind removing this? I think it's a leftover since printf was used for
logging some time ago.

[henrika (OOO until Aug 14), 2013/12/13 14:15:27]

Done.

   aos->Stop();
   ais->Stop();
   aos->Close();
   ais->Close();
 }
 
 }  // namespace media