| Index: content/renderer/media/audio_renderer_impl.cc
|
| ===================================================================
|
| --- content/renderer/media/audio_renderer_impl.cc (revision 123772)
|
| +++ content/renderer/media/audio_renderer_impl.cc (working copy)
|
| @@ -16,26 +16,6 @@
|
| #include "media/audio/audio_util.h"
|
| #include "media/base/filter_host.h"
|
|
|
| -// We define GetBufferSizeForSampleRate() instead of using
|
| -// GetAudioHardwareBufferSize() in audio_util because we're using
|
| -// the AUDIO_PCM_LINEAR flag, instead of AUDIO_PCM_LOW_LATENCY,
|
| -// which the audio_util functions assume.
|
| -//
|
| -// See: http://code.google.com/p/chromium/issues/detail?id=103627
|
| -// for a more detailed description of the subtleties.
|
| -static size_t GetBufferSizeForSampleRate(int sample_rate) {
|
| - // kNominalBufferSize has been tested on Windows, Mac OS X, and Linux
|
| - // using the low-latency audio codepath (SyncSocket implementation)
|
| - // with the AUDIO_PCM_LINEAR flag.
|
| - const size_t kNominalBufferSize = 2048;
|
| -
|
| - if (sample_rate <= 48000)
|
| - return kNominalBufferSize;
|
| - else if (sample_rate <= 96000)
|
| - return kNominalBufferSize * 2;
|
| - return kNominalBufferSize * 4;
|
| -}
|
| -
|
| AudioRendererImpl::AudioRendererImpl(media::AudioRendererSink* sink)
|
| : AudioRendererBase(),
|
| bytes_per_second_(0),
|
| @@ -91,7 +71,7 @@
|
|
|
| if (!is_initialized_) {
|
| sink_->Initialize(
|
| - GetBufferSizeForSampleRate(sample_rate),
|
| + media::SelectSamplesPerPacket(sample_rate),
|
| audio_parameters_.channels,
|
| audio_parameters_.sample_rate,
|
| audio_parameters_.format,
|
|
|