From d9514c1521f6a4b1bdba2eddfb8083a0a1d22be4 Mon Sep 17 00:00:00 2001 From: "vrk@google.com" Date: Tue, 5 Apr 2011 19:05:50 +0000 Subject: Fix valgrind messages for uninitialized preload_ value. The preload_ field was uninitialized when PipelineImpl was constructed, so PipelineImpl unit tests failed Valgrind tests. CL 80465 introduced the problem. BUG=NONE TEST=NONE TBR=acolwell Review URL: http://codereview.chromium.org/6720052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80503 0039d316-1c4b-4281-b951-d872f2087c98 --- media/base/pipeline_impl.cc | 1 + media/base/pipeline_impl_unittest.cc | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/media/base/pipeline_impl.cc b/media/base/pipeline_impl.cc index 2555bd7..f574b85 100644 --- a/media/base/pipeline_impl.cc +++ b/media/base/pipeline_impl.cc @@ -367,6 +367,7 @@ void PipelineImpl::ResetState() { video_width_ = 0; video_height_ = 0; volume_ = 1.0f; + preload_ = AUTO; playback_rate_ = 0.0f; status_ = PIPELINE_OK; has_audio_ = false; diff --git a/media/base/pipeline_impl_unittest.cc b/media/base/pipeline_impl_unittest.cc index 87f3e96..2028e7a 100644 --- a/media/base/pipeline_impl_unittest.cc +++ b/media/base/pipeline_impl_unittest.cc @@ -100,6 +100,7 @@ class PipelineImplTest : public ::testing::Test { mocks_->demuxer()->SetTotalAndBufferedBytesAndDuration( kTotalBytes, kBufferedBytes, duration); EXPECT_CALL(*mocks_->demuxer(), SetPlaybackRate(0.0f)); + EXPECT_CALL(*mocks_->demuxer(), SetPreload(AUTO)); EXPECT_CALL(*mocks_->demuxer(), Seek(base::TimeDelta(), NotNull())) .WillOnce(Invoke(&RunFilterCallback)); EXPECT_CALL(*mocks_->demuxer(), Stop(NotNull())) @@ -850,4 +851,3 @@ TEST(PipelineStatusNotificationTest, DelayedCallback) { } } // namespace media - -- cgit v1.1