diff options
| author | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-02 09:55:49 +0000 |
|---|---|---|
| committer | dalecurtis@chromium.org <dalecurtis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-02 09:55:49 +0000 |
| commit | c1674a529fb13fa97fa108a0dd5d7f93198a5e25 (patch) | |
| tree | ca42a015934f5bcd4d48ff0f40ebbf95d61eae44 /media/base/audio_discard_helper_unittest.cc | |
| parent | 7bd79e2ef968908125e7c5baf98e9be422eb39ea (diff) | |
| download | chromium_src-c1674a529fb13fa97fa108a0dd5d7f93198a5e25.zip chromium_src-c1674a529fb13fa97fa108a0dd5d7f93198a5e25.tar.gz chromium_src-c1674a529fb13fa97fa108a0dd5d7f93198a5e25.tar.bz2 | |
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
Diffstat (limited to 'media/base/audio_discard_helper_unittest.cc')
| -rw-r--r-- | media/base/audio_discard_helper_unittest.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/media/base/audio_discard_helper_unittest.cc b/media/base/audio_discard_helper_unittest.cc index 7788e7f..e3f21c0 100644 --- a/media/base/audio_discard_helper_unittest.cc +++ b/media/base/audio_discard_helper_unittest.cc @@ -33,7 +33,6 @@ static scoped_refptr<AudioBuffer> CreateDecodedBuffer(int frames) { 0.0f, kDataStep, frames, - kNoTimestamp(), kNoTimestamp()); } |