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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: media/base/video_frame_impl_unittest.cc

Issue 113611: Handle end of stream for media... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 7 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/video_frame_impl.cc ('k') | media/filters/audio_renderer_base.cc » ('j') | media/filters/video_thread.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/video_frame_impl_unittest.cc
===================================================================
--- media/base/video_frame_impl_unittest.cc (revision 17376)
+++ media/base/video_frame_impl_unittest.cc (working copy)
@@ -91,9 +91,6 @@
frame->SetDuration(kDurationB);
EXPECT_TRUE(kTimestampB == frame->GetTimestamp());
EXPECT_TRUE(kDurationB == frame->GetDuration());
- frame->SetEndOfStream(true);
- EXPECT_TRUE(frame->IsEndOfStream());
- frame->SetEndOfStream(false);
EXPECT_FALSE(frame->IsEndOfStream());
frame->SetDiscontinuous(true);
EXPECT_TRUE(frame->IsDiscontinuous());
@@ -105,4 +102,8 @@
ExpectFrameColor(frame, 0xFF000000);
media::MockVideoDecoder::InitializeYV12Frame(frame, 1.0f);
ExpectFrameColor(frame, 0xFFFFFFFF);
+
+ // Test an empty frame.
+ media::VideoFrameImpl::CreateEmptyFrame(&frame);
+ EXPECT_TRUE(frame->IsEndOfStream());
}
« no previous file with comments | « media/base/video_frame_impl.cc ('k') | media/filters/audio_renderer_base.cc » ('j') | media/filters/video_thread.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698