Remove AudioBuffer::set_duration(), instead base on frames.
commitc1674a529fb13fa97fa108a0dd5d7f93198a5e25
authordalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Fri, 2 May 2014 09:55:49 +0000 (2 09:55 +0000)
committerdalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Fri, 2 May 2014 09:55:49 +0000 (2 09:55 +0000)
treeca42a015934f5bcd4d48ff0f40ebbf95d61eae44
parent7bd79e2ef968908125e7c5baf98e9be422eb39ea
Remove AudioBuffer::set_duration(), instead base on frames.

wolenetz@ asked on https://codereview.chromium.org/251893002/ why
we allow set_duration() to be called.  It used to be required since
AudioBuffers had no concept of sample rate.  However, we now attach
a sample rate to every AudioBuffer which allows us to enforce a
valid duration for every buffer.

This CL also aligns MakeAudioBuffer() to do the same thing with
planar and interleaved data for the sake of easier testing.

BUG=none
TEST=media_unittests
NOTRY=true

Review URL: https://codereview.chromium.org/261533002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267779 0039d316-1c4b-4281-b951-d872f2087c98
18 files changed:
content/renderer/pepper/content_decryptor_delegate.cc
media/base/audio_buffer.cc
media/base/audio_buffer.h
media/base/audio_buffer_converter.cc
media/base/audio_buffer_converter_unittest.cc
media/base/audio_buffer_queue_unittest.cc
media/base/audio_buffer_unittest.cc
media/base/audio_discard_helper.cc
media/base/audio_discard_helper_unittest.cc
media/base/audio_splicer.cc
media/base/audio_splicer_unittest.cc
media/base/test_helpers.cc
media/base/test_helpers.h
media/filters/audio_renderer_algorithm_unittest.cc
media/filters/audio_renderer_impl_unittest.cc
media/filters/decrypting_audio_decoder.cc
media/filters/decrypting_audio_decoder_unittest.cc
media/filters/ffmpeg_audio_decoder_unittest.cc