Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(380)

Unified Diff: media/base/pipeline_unittest.cc

Issue 9718013: Merge FilterStatusCB into PipelineStatusCB. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/base/pipeline.cc ('k') | media/filters/audio_renderer_base.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/pipeline_unittest.cc
diff --git a/media/base/pipeline_unittest.cc b/media/base/pipeline_unittest.cc
index 83b3c822bcf683fb2bc73cb6b95babcd1c0247dc..bb25dbade77870e44fdd1fbe43984975fbf80b9f 100644
--- a/media/base/pipeline_unittest.cc
+++ b/media/base/pipeline_unittest.cc
@@ -106,7 +106,7 @@ class PipelineTest : public ::testing::Test {
EXPECT_CALL(*mocks_->demuxer(), SetPlaybackRate(0.0f));
EXPECT_CALL(*mocks_->demuxer(), SetPreload(AUTO));
EXPECT_CALL(*mocks_->demuxer(), Seek(mocks_->demuxer()->GetStartTime(), _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
EXPECT_CALL(*mocks_->demuxer(), Stop(_))
.WillOnce(Invoke(&RunStopFilterCallback));
@@ -134,7 +134,7 @@ class PipelineTest : public ::testing::Test {
EXPECT_CALL(*mocks_->video_decoder(), SetPlaybackRate(0.0f));
EXPECT_CALL(*mocks_->video_decoder(),
Seek(mocks_->demuxer()->GetStartTime(), _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
EXPECT_CALL(*mocks_->video_decoder(), Stop(_))
.WillOnce(Invoke(&RunStopFilterCallback));
}
@@ -153,7 +153,7 @@ class PipelineTest : public ::testing::Test {
EXPECT_CALL(*mocks_->video_renderer(), SetPlaybackRate(0.0f));
EXPECT_CALL(*mocks_->video_renderer(),
Seek(mocks_->demuxer()->GetStartTime(), _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
EXPECT_CALL(*mocks_->video_renderer(), Stop(_))
.WillOnce(Invoke(&RunStopFilterCallback));
}
@@ -173,7 +173,7 @@ class PipelineTest : public ::testing::Test {
EXPECT_CALL(*mocks_->audio_renderer(), SetPlaybackRate(0.0f));
EXPECT_CALL(*mocks_->audio_renderer(), SetVolume(1.0f));
EXPECT_CALL(*mocks_->audio_renderer(), Seek(base::TimeDelta(), _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
EXPECT_CALL(*mocks_->audio_renderer(), Stop(_))
.WillOnce(Invoke(&RunStopFilterCallback));
}
@@ -224,18 +224,18 @@ class PipelineTest : public ::testing::Test {
void ExpectSeek(const base::TimeDelta& seek_time) {
// Every filter should receive a call to Seek().
EXPECT_CALL(*mocks_->demuxer(), Seek(seek_time, _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
if (audio_stream_) {
EXPECT_CALL(*mocks_->audio_renderer(), Seek(seek_time, _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
}
if (video_stream_) {
EXPECT_CALL(*mocks_->video_decoder(), Seek(seek_time, _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
EXPECT_CALL(*mocks_->video_renderer(), Seek(seek_time, _))
- .WillOnce(Invoke(&RunFilterStatusCB));
+ .WillOnce(Invoke(&RunPipelineStatusCB));
}
// We expect a successful seek callback.
@@ -726,7 +726,7 @@ TEST_F(PipelineTest, AudioStreamShorterThanVideo) {
host->NotifyEnded();
}
-void SendReadErrorToCB(::testing::Unused, const FilterStatusCB& cb) {
+void SendReadErrorToCB(::testing::Unused, const PipelineStatusCB& cb) {
cb.Run(PIPELINE_ERROR_READ);
}
« no previous file with comments | « media/base/pipeline.cc ('k') | media/filters/audio_renderer_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698