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

Unified Diff: chromeos/audio/audio_devices_pref_handler_impl_unittest.cc

Issue 1380103003: Store audio device's active state in preference (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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: chromeos/audio/audio_devices_pref_handler_impl_unittest.cc
diff --git a/chromeos/audio/audio_devices_pref_handler_impl_unittest.cc b/chromeos/audio/audio_devices_pref_handler_impl_unittest.cc
index ba871325debb5cd4c47a626a676e7023f11d4130..bebf7e39f5e46b035cef0391c12a38aea22ad710 100644
--- a/chromeos/audio/audio_devices_pref_handler_impl_unittest.cc
+++ b/chromeos/audio/audio_devices_pref_handler_impl_unittest.cc
@@ -22,6 +22,7 @@ const uint64 kOtherTypeInputId = 90002;
const AudioDevice kInternalMic(AudioNode(true,
kInternalMicId,
+ kInternalMicId,
"Fake Mic",
"INTERNAL_MIC",
"Internal Mic",
@@ -30,6 +31,7 @@ const AudioDevice kInternalMic(AudioNode(true,
const AudioDevice kHeadphone(AudioNode(false,
kHeadphoneId,
+ kHeadphoneId,
"Fake Headphone",
"HEADPHONE",
"Headphone",
@@ -38,6 +40,7 @@ const AudioDevice kHeadphone(AudioNode(false,
const AudioDevice kHDMIOutput(AudioNode(false,
kHDMIOutputId,
+ kHDMIOutputId,
"HDMI output",
"HDMI",
"HDMI output",
@@ -47,6 +50,7 @@ const AudioDevice kHDMIOutput(AudioNode(false,
const AudioDevice kInputDeviceWithSpecialCharacters(
AudioNode(true,
kOtherTypeInputId,
+ kOtherTypeInputId,
"Fake ~!@#$%^&*()_+`-=<>?,./{}|[]\\\\Mic",
"SOME_OTHER_TYPE",
"Other Type Input Device",
@@ -56,6 +60,7 @@ const AudioDevice kInputDeviceWithSpecialCharacters(
const AudioDevice kOutputDeviceWithSpecialCharacters(
AudioNode(false,
kOtherTypeOutputId,
+ kOtherTypeOutputId,
"Fake ~!@#$%^&*()_+`-=<>?,./{}|[]\\\\Headphone",
"SOME_OTHER_TYPE",
"Other Type Output Device",
@@ -89,6 +94,12 @@ TEST_F(AudioDevicesPrefHandlerTest, TestDefaultValues) {
EXPECT_EQ(75.0, audio_pref_handler_->GetInputGainValue(&kInternalMic));
EXPECT_EQ(75.0, audio_pref_handler_->GetOutputVolumeValue(&kHeadphone));
EXPECT_EQ(75.0, audio_pref_handler_->GetOutputVolumeValue(&kHDMIOutput));
+ EXPECT_EQ(AUDIO_STATE_NOT_AVAILABLE,
+ audio_pref_handler_->GetDeviceState(kInternalMic));
+ EXPECT_EQ(AUDIO_STATE_NOT_AVAILABLE,
+ audio_pref_handler_->GetDeviceState(kHeadphone));
+ EXPECT_EQ(AUDIO_STATE_NOT_AVAILABLE,
+ audio_pref_handler_->GetDeviceState(kHDMIOutput));
}
TEST_F(AudioDevicesPrefHandlerTest, PrefsRegistered) {
@@ -98,6 +109,7 @@ TEST_F(AudioDevicesPrefHandlerTest, PrefsRegistered) {
EXPECT_TRUE(pref_service_->FindPreference(prefs::kAudioOutputAllowed));
EXPECT_TRUE(pref_service_->FindPreference(prefs::kAudioVolumePercent));
EXPECT_TRUE(pref_service_->FindPreference(prefs::kAudioMute));
+ EXPECT_TRUE(pref_service_->FindPreference(prefs::kAudioDevicesState));
}
TEST_F(AudioDevicesPrefHandlerTest, TestBasicInputOutputDevices) {

Powered by Google App Engine
This is Rietveld 408576698