diff options
author | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-29 01:36:45 +0000 |
---|---|---|
committer | dalecurtis@google.com <dalecurtis@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-29 01:36:45 +0000 |
commit | da67d5d7cf16c47a2778519be4db2c72d2838de0 (patch) | |
tree | dc239af4994de22ce41c03b7c34f00a9dbc8baef /media/base/audio_buffer_converter_unittest.cc | |
parent | b8fc82b108137edd95e0bdbc1f551caf37edac8e (diff) | |
download | chromium_src-da67d5d7cf16c47a2778519be4db2c72d2838de0.zip chromium_src-da67d5d7cf16c47a2778519be4db2c72d2838de0.tar.gz chromium_src-da67d5d7cf16c47a2778519be4db2c72d2838de0.tar.bz2 |
Make sure AudioBufferConverter uses aligned AudioBus channels.
SIMD requires aligned loads or it'll crash.
BUG=357840
TEST=new unittest. played video back in crashes.
R=rileya@chromium.org, scherkus@chromium.org
TBR=scherkus
Review URL: https://codereview.chromium.org/217923002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260336 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base/audio_buffer_converter_unittest.cc')
-rw-r--r-- | media/base/audio_buffer_converter_unittest.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/media/base/audio_buffer_converter_unittest.cc b/media/base/audio_buffer_converter_unittest.cc index 032702e..77e787e 100644 --- a/media/base/audio_buffer_converter_unittest.cc +++ b/media/base/audio_buffer_converter_unittest.cc @@ -12,6 +12,8 @@ namespace media { +// Important: Use an odd buffer size here so SIMD issues are caught. +const int kOutFrameSize = 441; const int kOutSampleRate = 44100; const ChannelLayout kOutChannelLayout = CHANNEL_LAYOUT_STEREO; const int kOutChannelCount = 2; @@ -39,7 +41,7 @@ class AudioBufferConverterTest : public ::testing::Test { kOutChannelLayout, kOutSampleRate, 16, - 512); + kOutFrameSize); ResetConverter(output_params); } |