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

Unified Diff: media/audio/audio_manager_base.cc

Issue 1892433002: Moving device description utils from AudioManager[Base] into AudioDeviceDescription; to be shared b… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: another rebase Created 4 years, 8 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/audio_manager_base.h ('k') | media/audio/audio_manager_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/audio_manager_base.cc
diff --git a/media/audio/audio_manager_base.cc b/media/audio/audio_manager_base.cc
index 47ca23d18f680951a30e2e9563e9dffc70f102c6..5c3e042e9ffd0d584b40ea36c611b63f7d9a0c79 100644
--- a/media/audio/audio_manager_base.cc
+++ b/media/audio/audio_manager_base.cc
@@ -12,6 +12,7 @@
#include "base/strings/string_number_conversions.h"
#include "base/thread_task_runner_handle.h"
#include "build/build_config.h"
+#include "media/audio/audio_device_description.h"
#include "media/audio/audio_output_dispatcher_impl.h"
#include "media/audio/audio_output_proxy.h"
#include "media/audio/audio_output_resampler.h"
@@ -36,10 +37,6 @@ const int kMaxInputChannels = 3;
} // namespace
-const char AudioManagerBase::kDefaultDeviceId[] = "default";
-const char AudioManagerBase::kCommunicationsDeviceId[] = "communications";
-const char AudioManagerBase::kLoopbackInputDeviceId[] = "loopback";
-
struct AudioManagerBase::DispatcherParams {
DispatcherParams(const AudioParameters& input,
const AudioParameters& output,
@@ -77,18 +74,6 @@ class AudioManagerBase::CompareByParams {
const DispatcherParams* dispatcher_;
};
-// static
-bool AudioManagerBase::IsDefaultDeviceId(const std::string& device_id) {
- return device_id.empty() || device_id == AudioManagerBase::kDefaultDeviceId;
-}
-
-// static
-bool AudioManagerBase::UseSessionIdToSelectDevice(
- int session_id,
- const std::string& device_id) {
- return session_id && device_id.empty();
-}
-
AudioManagerBase::AudioManagerBase(
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
scoped_refptr<base::SingleThreadTaskRunner> worker_task_runner,
@@ -145,7 +130,7 @@ AudioOutputStream* AudioManagerBase::MakeAudioOutputStream(
AudioOutputStream* stream;
switch (params.format()) {
case AudioParameters::AUDIO_PCM_LINEAR:
- DCHECK(IsDefaultDeviceId(device_id))
+ DCHECK(AudioDeviceDescription::IsDefaultDevice(device_id))
<< "AUDIO_PCM_LINEAR supports only the default device.";
stream = MakeLinearOutputStream(params);
break;
@@ -225,7 +210,9 @@ AudioOutputStream* AudioManagerBase::MakeAudioOutputStreamProxy(
// NOTE: Implementations that don't yet support opening non-default output
// devices may return an empty string from GetDefaultOutputDeviceID().
std::string output_device_id =
- IsDefaultDeviceId(device_id) ? GetDefaultOutputDeviceID() : device_id;
+ AudioDeviceDescription::IsDefaultDevice(device_id)
+ ? GetDefaultOutputDeviceID()
+ : device_id;
// If we're not using AudioOutputResampler our output parameters are the same
// as our input parameters.
« no previous file with comments | « media/audio/audio_manager_base.h ('k') | media/audio/audio_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698