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

Unified Diff: media/audio/linux/audio_manager_linux.cc

Issue 9655018: Make AudioParameters a class instead of a struct (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix tests Created 8 years, 9 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/linux/audio_manager_linux.cc
diff --git a/media/audio/linux/audio_manager_linux.cc b/media/audio/linux/audio_manager_linux.cc
index da24f1228d30a3afda012d08f1a1ca33f0e161b8..66e794680456586e4d690866e9240e06cb82b102 100644
--- a/media/audio/linux/audio_manager_linux.cc
+++ b/media/audio/linux/audio_manager_linux.cc
@@ -50,7 +50,7 @@ bool AudioManagerLinux::HasAudioInputDevices() {
AudioOutputStream* AudioManagerLinux::MakeAudioOutputStream(
const AudioParameters& params) {
// Early return for testing hook.
- if (params.format == AudioParameters::AUDIO_MOCK)
+ if (params.format() == AudioParameters::AUDIO_MOCK)
return FakeAudioOutputStream::MakeFakeStream(params);
// Don't allow opening more than |kMaxOutputStreams| streams.
@@ -80,12 +80,12 @@ AudioOutputStream* AudioManagerLinux::MakeAudioOutputStream(
AudioInputStream* AudioManagerLinux::MakeAudioInputStream(
const AudioParameters& params, const std::string& device_id) {
- if (!params.IsValid() || params.channels > kMaxInputChannels ||
+ if (!params.IsValid() || params.channels() > kMaxInputChannels ||
device_id.empty()) {
return NULL;
}
- if (params.format == AudioParameters::AUDIO_MOCK) {
+ if (params.format() == AudioParameters::AUDIO_MOCK) {
return FakeAudioInputStream::MakeFakeStream(params);
}

Powered by Google App Engine
This is Rietveld 408576698