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

Side by Side Diff: media/audio/linux/audio_manager_linux.cc

Issue 10937027: Reland 10952007: Pass through small buffer sizes without FIFO on Linux (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « media/audio/linux/audio_manager_linux.h ('k') | media/audio/win/audio_manager_win.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/audio/linux/audio_manager_linux.h" 5 #include "media/audio/linux/audio_manager_linux.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/environment.h" 8 #include "base/environment.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/nix/xdg_util.h" 10 #include "base/nix/xdg_util.h"
11 #include "base/process_util.h" 11 #include "base/process_util.h"
12 #include "base/stl_util.h" 12 #include "base/stl_util.h"
13 #include "media/audio/audio_output_dispatcher.h" 13 #include "media/audio/audio_output_dispatcher.h"
14 #include "media/audio/audio_util.h"
14 #include "media/audio/linux/alsa_input.h" 15 #include "media/audio/linux/alsa_input.h"
15 #include "media/audio/linux/alsa_output.h" 16 #include "media/audio/linux/alsa_output.h"
16 #include "media/audio/linux/alsa_wrapper.h" 17 #include "media/audio/linux/alsa_wrapper.h"
17 #if defined(USE_PULSEAUDIO) 18 #if defined(USE_PULSEAUDIO)
18 #include "media/audio/pulse/pulse_output.h" 19 #include "media/audio/pulse/pulse_output.h"
19 #endif 20 #endif
20 #if defined(USE_CRAS) 21 #if defined(USE_CRAS)
21 #include "media/audio/linux/cras_input.h" 22 #include "media/audio/linux/cras_input.h"
22 #include "media/audio/linux/cras_output.h" 23 #include "media/audio/linux/cras_output.h"
23 #endif 24 #endif
(...skipping 311 matching lines...) Expand 10 before | Expand all | Expand 10 after
335 switches::kAlsaInputDevice); 336 switches::kAlsaInputDevice);
336 } 337 }
337 338
338 return new AlsaPcmInputStream(this, device_name, params, wrapper_.get()); 339 return new AlsaPcmInputStream(this, device_name, params, wrapper_.get());
339 } 340 }
340 341
341 AudioManager* CreateAudioManager() { 342 AudioManager* CreateAudioManager() {
342 return new AudioManagerLinux(); 343 return new AudioManagerLinux();
343 } 344 }
344 345
346 AudioParameters AudioManagerLinux::GetPreferredLowLatencyOutputStreamParameters(
347 const AudioParameters& input_params) {
348 // Since Linux doesn't actually have a low latency path the hardware buffer
349 // size is quite large in order to prevent glitches with general usage. Some
350 // clients, such as WebRTC, have a more limited use case and work acceptably
351 // with a smaller buffer size. The check below allows clients which want to
352 // try a smaller buffer size on Linux to do so.
353 int buffer_size = 0;
354 if (static_cast<size_t>(input_params.frames_per_buffer()) <
355 GetAudioHardwareBufferSize()) {
356 buffer_size = input_params.frames_per_buffer();
357 } else {
358 buffer_size = GetAudioHardwareBufferSize();
359 }
360
361 // TODO(dalecurtis): This should include bits per channel and channel layout
362 // eventually.
363 return AudioParameters(
364 AudioParameters::AUDIO_PCM_LOW_LATENCY, input_params.channel_layout(),
365 GetAudioHardwareSampleRate(), 16, buffer_size);
366 }
367
345 } // namespace media 368 } // namespace media
OLDNEW
« no previous file with comments | « media/audio/linux/audio_manager_linux.h ('k') | media/audio/win/audio_manager_win.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698