Index: media/audio/linux/alsa_output.cc |
diff --git a/media/audio/linux/alsa_output.cc b/media/audio/linux/alsa_output.cc |
index 338ff082a92d4d44a526dd24125aed23679aa0f1..d8e502cf7699b60b49c794be0faa2c289f8b0408 100644 |
--- a/media/audio/linux/alsa_output.cc |
+++ b/media/audio/linux/alsa_output.cc |
@@ -1,4 +1,4 @@ |
-// Copyright (c) 2010 The Chromium Authors. All rights reserved. |
+// Copyright (c) 2011 The Chromium Authors. All rights reserved. |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
// |
@@ -105,20 +105,9 @@ static const int kPcmRecoverIsSilent = 1; |
static const int kPcmRecoverIsSilent = 0; |
#endif |
-const char AlsaPcmOutputStream::kDefaultDevice[] = "default"; |
-const char AlsaPcmOutputStream::kAutoSelectDevice[] = ""; |
-const char AlsaPcmOutputStream::kPlugPrefix[] = "plug:"; |
- |
-// Since we expect to only be able to wake up with a resolution of |
-// kSleepErrorMilliseconds, double that for our minimum required latency. |
-const uint32 AlsaPcmOutputStream::kMinLatencyMicros = |
- kSleepErrorMilliseconds * 2 * 1000; |
- |
-namespace { |
- |
// ALSA is currently limited to 48Khz. |
// TODO(fbarchard): Resample audio from higher frequency to 48000. |
-const int kAlsaMaxSampleRate = 48000; |
+static const int kAlsaMaxSampleRate = 48000; |
// While the "default" device may support multi-channel audio, in Alsa, only |
// the device names surround40, surround41, surround50, etc, have a defined |
@@ -137,7 +126,7 @@ const int kAlsaMaxSampleRate = 48000; |
// TODO(ajwong): The source data should have enough info to tell us if we want |
// surround41 versus surround51, etc., instead of needing us to guess base don |
// channel number. Fix API to pass that data down. |
-const char* GuessSpecificDeviceName(uint32 channels) { |
+static const char* GuessSpecificDeviceName(uint32 channels) { |
switch (channels) { |
case 8: |
return "surround71"; |
@@ -190,10 +179,6 @@ static void Swizzle51Layout(Format* b, uint32 filled) { |
} |
} |
-} // namespace |
- |
-// Not in an anonymous namespace so that it can be a friend to |
-// AlsaPcmOutputStream. |
std::ostream& operator<<(std::ostream& os, |
AlsaPcmOutputStream::InternalState state) { |
switch (state) { |
@@ -219,6 +204,15 @@ std::ostream& operator<<(std::ostream& os, |
return os; |
} |
+const char AlsaPcmOutputStream::kDefaultDevice[] = "default"; |
+const char AlsaPcmOutputStream::kAutoSelectDevice[] = ""; |
+const char AlsaPcmOutputStream::kPlugPrefix[] = "plug:"; |
+ |
+// Since we expect to only be able to wake up with a resolution of |
+// kSleepErrorMilliseconds, double that for our minimum required latency. |
+const uint32 AlsaPcmOutputStream::kMinLatencyMicros = |
+ kSleepErrorMilliseconds * 2 * 1000; |
+ |
AlsaPcmOutputStream::AlsaPcmOutputStream(const std::string& device_name, |
AudioParameters params, |
AlsaWrapper* wrapper, |