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

Unified Diff: chromecast/media/cma/backend/audio_video_pipeline_device_unittest.cc

Issue 2612353003: [Chromecast] Fix AudioVideoPipelineDeviceTest failures (Closed)
Patch Set: Created 3 years, 11 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
Index: chromecast/media/cma/backend/audio_video_pipeline_device_unittest.cc
diff --git a/chromecast/media/cma/backend/audio_video_pipeline_device_unittest.cc b/chromecast/media/cma/backend/audio_video_pipeline_device_unittest.cc
index a912145e0a7f85a7dc684637745d8d1efc73056d..7418f1f1b6135054ce5424700d90fa72adebd355 100644
--- a/chromecast/media/cma/backend/audio_video_pipeline_device_unittest.cc
+++ b/chromecast/media/cma/backend/audio_video_pipeline_device_unittest.cc
@@ -395,7 +395,9 @@ void BufferFeeder::OnPushBufferComplete(BufferStatus status) {
EXPECT_TRUE(expecting_buffer_complete_)
<< "OnPushBufferComplete() called unexpectedly";
expecting_buffer_complete_ = false;
- ASSERT_NE(status, MediaPipelineBackend::kBufferFailed);
+ if (!feed_continuous_pcm_ || !feeding_completed_) {
+ ASSERT_NE(status, MediaPipelineBackend::kBufferFailed);
+ }
EXPECT_FALSE(eos_) << "Got OnPushBufferComplete() after OnEndOfStream()";
if (test_config_after_next_push_) {

Powered by Google App Engine
This is Rietveld 408576698