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

Unified Diff: media/audio/cras/audio_manager_cras.cc

Issue 1362093002: Re-land: Add localized default audio device names. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Exclude media_resource_provider on iOS. Created 5 years, 3 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
« no previous file with comments | « media/audio/cras/audio_manager_cras.h ('k') | media/audio/mac/audio_manager_mac.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/cras/audio_manager_cras.cc
diff --git a/media/audio/cras/audio_manager_cras.cc b/media/audio/cras/audio_manager_cras.cc
index 2b47de40cc9f7d04c9525000bdc3ba2a32efa2c7..fc58004e2488ff84259ae60191d47627b2a69b87 100644
--- a/media/audio/cras/audio_manager_cras.cc
+++ b/media/audio/cras/audio_manager_cras.cc
@@ -16,6 +16,7 @@
#include "media/audio/cras/cras_input.h"
#include "media/audio/cras/cras_unified.h"
#include "media/base/channel_layout.h"
+#include "media/base/media_resources.h"
// cras_util.h headers pull in min/max macros...
// TODO(dgreid): Fix headers such that these aren't imported.
@@ -38,32 +39,27 @@ const int kMaximumOutputBufferSize = 8192;
// Default input buffer size.
const int kDefaultInputBufferSize = 1024;
+const char kBeamformingOnUniqueId[] = "beamforming-on";
+const char kBeamformingOffUniqueId[] = "beamforming-off";
+
void AddDefaultDevice(AudioDeviceNames* device_names) {
+ DCHECK(device_names->empty());
+
// Cras will route audio from a proper physical device automatically.
- device_names->push_back(
- AudioDeviceName(AudioManagerBase::kDefaultDeviceName,
- AudioManagerBase::kDefaultDeviceId));
+ device_names->push_back(AudioDeviceName(AudioManager::GetDefaultDeviceName(),
+ AudioManagerBase::kDefaultDeviceId));
}
-// Returns the AudioDeviceName of the virtual device with beamforming on.
-AudioDeviceName BeamformingOnDeviceName() {
- // TODO(ajm): Replace these strings with properly localized ones.
- // (crbug.com/497001)
- static const char kBeamformingOnNameSuffix[] = " (pick up just one person)";
- static const char kBeamformingOnIdSuffix[] = "-beamforming";
-
- return AudioDeviceName(
- std::string(AudioManagerBase::kDefaultDeviceName) +
- kBeamformingOnNameSuffix,
- std::string(AudioManagerBase::kDefaultDeviceId) + kBeamformingOnIdSuffix);
-}
+// Adds the beamforming on and off devices to |device_names|.
+void AddBeamformingDevices(AudioDeviceNames* device_names) {
+ DCHECK(device_names->empty());
-// Returns the AudioDeviceName of the virtual device with beamforming off.
-AudioDeviceName BeamformingOffDeviceName() {
- static const char kBeamformingOffNameSuffix[] = " (pick up everything)";
- return AudioDeviceName(std::string(AudioManagerBase::kDefaultDeviceName) +
- kBeamformingOffNameSuffix,
- AudioManagerBase::kDefaultDeviceId);
+ device_names->push_back(AudioDeviceName(
+ GetLocalizedStringUTF8(BEAMFORMING_ON_DEFAULT_AUDIO_INPUT_DEVICE_NAME),
+ kBeamformingOnUniqueId));
+ device_names->push_back(AudioDeviceName(
+ GetLocalizedStringUTF8(BEAMFORMING_OFF_DEFAULT_AUDIO_INPUT_DEVICE_NAME),
+ kBeamformingOffUniqueId));
}
// Returns a mic positions string if the machine has a beamforming capable
@@ -99,10 +95,7 @@ bool AudioManagerCras::HasAudioInputDevices() {
}
AudioManagerCras::AudioManagerCras(AudioLogFactory* audio_log_factory)
- : AudioManagerBase(audio_log_factory),
- has_keyboard_mic_(false),
- beamforming_on_device_name_(BeamformingOnDeviceName()),
- beamforming_off_device_name_(BeamformingOffDeviceName()) {
+ : AudioManagerBase(audio_log_factory), has_keyboard_mic_(false) {
SetMaxOutputStreamsAllowed(kMaxOutputStreams);
}
@@ -122,18 +115,14 @@ void AudioManagerCras::GetAudioInputDeviceNames(
// At least two mic positions indicates we have a beamforming capable mic
// array. Add the virtual beamforming device to the list. When this device is
// queried through GetInputStreamParameters, provide the cached mic positions.
- if (mic_positions_.size() > 1) {
- device_names->push_back(beamforming_on_device_name_);
- device_names->push_back(beamforming_off_device_name_);
- } else {
+ if (mic_positions_.size() > 1)
+ AddBeamformingDevices(device_names);
+ else
AddDefaultDevice(device_names);
- }
}
void AudioManagerCras::GetAudioOutputDeviceNames(
AudioDeviceNames* device_names) {
- DCHECK(device_names->empty());
-
AddDefaultDevice(device_names);
}
@@ -152,7 +141,7 @@ AudioParameters AudioManagerCras::GetInputStreamParameters(
buffer_size);
if (has_keyboard_mic_)
params.set_effects(AudioParameters::KEYBOARD_MIC);
- if (device_id == beamforming_on_device_name_.unique_id)
+ if (device_id == kBeamformingOnUniqueId)
params.set_mic_positions(mic_positions_);
return params;
}
« no previous file with comments | « media/audio/cras/audio_manager_cras.h ('k') | media/audio/mac/audio_manager_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698