Verify that AudioInputDeviceManage stores valid parameters if there are no audio devices in the sys…

content/browser/renderer_host/media/audio_input_device_manager_unittest.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 "content/browser/renderer_host/media/audio_input_device_manager.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/run_loop.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "build/build_config.h"
 #include "content/public/common/media_stream_request.h"
 #include "content/public/test/test_browser_thread_bundle.h"
-#include "media/audio/audio_manager_base.h"
 #include "media/audio/audio_system_impl.h"
 #include "media/audio/audio_thread_impl.h"
+#include "media/audio/mock_audio_manager.h"
 #include "media/base/media_switches.h"
 #include "media/base/test_helpers.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using testing::InSequence;
 
 namespace content {
 
 class MockAudioInputDeviceManagerListener
@@ skipping 21 matching lines @@
 #define MAYBE_AudioInputDeviceManagerTest DISABLED_AudioInputDeviceManagerTest
 #else
 #define MAYBE_AudioInputDeviceManagerTest AudioInputDeviceManagerTest
 #endif
 
 class MAYBE_AudioInputDeviceManagerTest : public testing::Test {
  public:
   MAYBE_AudioInputDeviceManagerTest() {}
 
  protected:
-  void SetUp() override {
+  virtual void Initialize() {
     base::CommandLine::ForCurrentProcess()->AppendSwitch(
         switches::kUseFakeDeviceForMediaStream);
     // AudioInputDeviceManager accesses AudioSystem from IO thread, so it never
     // runs on the same thread with it, even on Mac.
     audio_manager_ = media::AudioManager::CreateForTesting(
         base::MakeUnique<media::AudioThreadImpl>());
     // Flush the message loop to ensure proper initialization of AudioManager.
     base::RunLoop().RunUntilIdle();
 
+    // Use fake devices.
+    devices_.emplace_back(MEDIA_DEVICE_AUDIO_CAPTURE, "Fake Device 1",
+                          "fake_device_1");
+    devices_.emplace_back(MEDIA_DEVICE_AUDIO_CAPTURE, "Fake Device 2",
+                          "fake_device_2");
+  }
+
+  void SetUp() override {
+    Initialize();
+
     audio_system_ = media::AudioSystemImpl::Create(audio_manager_.get());
     manager_ = new AudioInputDeviceManager(audio_system_.get());
     audio_input_listener_.reset(new MockAudioInputDeviceManagerListener());
     manager_->RegisterListener(audio_input_listener_.get());
 
-    // Use fake devices.
-    devices_.emplace_back(MEDIA_DEVICE_AUDIO_CAPTURE, "Fake Device 1",
-                          "fake_device_1");
-    devices_.emplace_back(MEDIA_DEVICE_AUDIO_CAPTURE, "Fake Device 2",
-                          "fake_device_2");
-
     // Wait until we get the list.
     base::RunLoop().RunUntilIdle();
   }
 
   void TearDown() override {
     manager_->UnregisterListener(audio_input_listener_.get());
     audio_manager_->Shutdown();
   }
 
   void WaitForOpenCompletion() {
@@ skipping 191 matching lines @@
       manager_->GetOpenedDeviceInfoById(invalid_session_id);
   DCHECK(!info);
 
   manager_->Close(session_id);
   EXPECT_CALL(*audio_input_listener_,
               Closed(MEDIA_DEVICE_AUDIO_CAPTURE, session_id))
       .Times(1);
   base::RunLoop().RunUntilIdle();
 }
 
+class AudioInputDeviceManagerNoDevicesTest
+    : public MAYBE_AudioInputDeviceManagerTest {
+ public:
+  AudioInputDeviceManagerNoDevicesTest(){};
+
+ protected:
+  void Initialize() override {
+    audio_manager_ = base::MakeUnique<media::MockAudioManager>(
+        base::MakeUnique<media::AudioThreadImpl>());
+
+    devices_.emplace_back(MEDIA_TAB_AUDIO_CAPTURE, "Tab capture",

[Guido Urdaneta, 2017/07/05 09:37:12]

If this is about no devices, why are there three devices here, including a
MEDIA_DEVICE_AUDIO_CAPTURE one?

Also, can this be done as just another test in AudioInputDeviceManagerTest?

[o1ka, 2017/07/05 10:10:31]

There are no physical devices: we use a mock audio manager.
(See the bug description which refers to
https://bugs.chromium.org/p/chromium/issues/detail?id=733356)
Because we are mocking absence of physical devices, and
AudioInputDeviceManagerTest uses fake devices.

[Guido Urdaneta, 2017/07/05 13:28:32]

Add a comment in the initialization of |audio_manager_| saying that this mock
manager simulates a manager without devices.

Also explain that you use |devices_| as application-provided (not
manager-controlled) devices. Also consider renaming |devices_| since my first
thought was that it was a list of devices managed by |audio_manager_|. Even
better, don't use |devices_| at all in Initialize() and just define an
array/vector of devices in ParametersValidWithoutAudioDevices.

Optional: rename |manager_| to |audio_input_device_manager_| since it is not the
only manager in the test.

+                          "tab_capture");
+
+    devices_.emplace_back(MEDIA_DESKTOP_AUDIO_CAPTURE, "Desktop capture",
+                          "desktop_capture");
+
+    devices_.emplace_back(MEDIA_DEVICE_AUDIO_CAPTURE, "Fake Device",
+                          "fake_device");
+  }
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(AudioInputDeviceManagerNoDevicesTest);
+};
+
+TEST_F(AudioInputDeviceManagerNoDevicesTest,
+       ParametersValidWithoutAudioDevices) {
+  ASSERT_FALSE(devices_.empty());
+
+  InSequence s;
+
+  for (StreamDeviceInfoArray::const_iterator iter = devices_.begin();
+       iter != devices_.end(); ++iter) {
+    // Opens/closes the devices.
+    int session_id = manager_->Open(iter->device);
+
+    // Expected mock call with expected return value.
+    EXPECT_CALL(*audio_input_listener_, Opened(iter->device.type, session_id))
+        .Times(1);
+    // Waits for the callback.
+    WaitForOpenCompletion();
+
+    // Expects that device parameters stored by the manager are valid.
+    const StreamDeviceInfo* device_info =
+        manager_->GetOpenedDeviceInfoById(session_id);
+    EXPECT_TRUE(
+        media::AudioParameters(media::AudioParameters::AUDIO_FAKE,
+                               static_cast<media::ChannelLayout>(
+                                   device_info->device.input.channel_layout),
+                               device_info->device.input.sample_rate, 16,
+                               device_info->device.input.frames_per_buffer)
+            .IsValid());
+
+    manager_->Close(session_id);
+    EXPECT_CALL(*audio_input_listener_, Closed(iter->device.type, session_id))
+        .Times(1);
+
+    // Waits for the callback.
+    base::RunLoop().RunUntilIdle();
+  }
+}
+
 }  // namespace content