diff options
| author | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-20 23:38:15 +0000 |
|---|---|---|
| committer | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-20 23:38:15 +0000 |
| commit | 74265f1015752e3769edaf843b1a2e26d4679aba (patch) | |
| tree | 4f685687e02484da9f304016bb4d01b5bc041a2a /media/base/audio_bus_unittest.cc | |
| parent | ede53d2cee1b5f9f86563a293e1ecfa8872fb64f (diff) | |
| download | chromium_src-74265f1015752e3769edaf843b1a2e26d4679aba.zip chromium_src-74265f1015752e3769edaf843b1a2e26d4679aba.tar.gz chromium_src-74265f1015752e3769edaf843b1a2e26d4679aba.tar.bz2 | |
Fix incorrect truncation of audio data during partial interleave.
Loop should run from [start_frame, start_frame + frames), not
[start_frame, frames). As luck would have it, we even have a unit
test for this... but it was also wrong for a different reason :x
BUG=176624
TEST=fixed unitest.
Review URL: https://chromiumcodereview.appspot.com/12317027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@183656 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/base/audio_bus_unittest.cc')
| -rw-r--r-- | media/base/audio_bus_unittest.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/base/audio_bus_unittest.cc b/media/base/audio_bus_unittest.cc index affce35..8c3a59f 100644 --- a/media/base/audio_bus_unittest.cc +++ b/media/base/audio_bus_unittest.cc @@ -353,7 +353,7 @@ TEST_F(AudioBusTest, ToInterleavedPartial) { kPartialStart, kPartialFrames, sizeof(*kTestVectorInt16), test_array); ASSERT_EQ(memcmp( test_array, kTestVectorInt16 + kPartialStart * kTestVectorChannels, - kPartialFrames * sizeof(*kTestVectorInt16)), 0); + kPartialFrames * sizeof(*kTestVectorInt16) * kTestVectorChannels), 0); } } // namespace media |