Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(140)

Unified Diff: media/audio/audio_system_impl_unittest.cc

Issue 2687853002: Switching VirtualKeyboardPrivate keyboard config call stack to receive HasInputDevices() responce a… (Closed)
Patch Set: Fix for compilation error on linux_chromium_chromeos Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/audio/audio_system_impl_unittest.cc
diff --git a/media/audio/audio_system_impl_unittest.cc b/media/audio/audio_system_impl_unittest.cc
index 3247cddbd04fccfa56b8a6a686816896b787276b..04f6f645a6752f44d43d9d2af910f8fbb53159ba 100644
--- a/media/audio/audio_system_impl_unittest.cc
+++ b/media/audio/audio_system_impl_unittest.cc
@@ -33,10 +33,13 @@ class AudioSystemImplTest : public testing::TestWithParam<bool> {
audio_manager_->SetInputStreamParameters(
media::AudioParameters::UnavailableDeviceParams());
audio_system_ = media::AudioSystemImpl::Create(audio_manager_.get());
+ EXPECT_EQ(AudioSystem::Get(), audio_system_.get());
}
~AudioSystemImplTest() override {
// Deleting |audio_manager_| on its thread.
+ audio_system_.reset();
+ EXPECT_EQ(AudioSystem::Get(), nullptr);
audio_manager_.reset();
audio_thread_.Stop();
}
@@ -65,6 +68,7 @@ class AudioSystemImplTest : public testing::TestWithParam<bool> {
}
MOCK_METHOD0(AudioParametersReceived, void(void));
+ MOCK_METHOD1(HasInputDevicesCallback, void(bool));
protected:
base::MessageLoop message_loop_;
@@ -94,6 +98,21 @@ TEST_P(AudioSystemImplTest, GetInputStreamParametersNoDevice) {
WaitForCallback();
}
+TEST_P(AudioSystemImplTest, HasInputDevices) {
+ EXPECT_CALL(*this, HasInputDevicesCallback(true));
+ audio_system_->HasInputDevices(base::Bind(
+ &AudioSystemImplTest::HasInputDevicesCallback, base::Unretained(this)));
+ WaitForCallback();
+}
+
+TEST_P(AudioSystemImplTest, HasNoInputDevices) {
+ audio_manager_->SetHasInputDevices(false);
+ EXPECT_CALL(*this, HasInputDevicesCallback(false));
+ audio_system_->HasInputDevices(base::Bind(
+ &AudioSystemImplTest::HasInputDevicesCallback, base::Unretained(this)));
+ WaitForCallback();
+}
+
INSTANTIATE_TEST_CASE_P(, AudioSystemImplTest, testing::Values(false, true));
} // namespace media

Powered by Google App Engine
This is Rietveld 408576698