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

Unified Diff: media/filters/ffmpeg_demuxer_unittest.cc

Issue 363813002: Update to Pipeline Metadata and Decoder Stream for Orientation Data (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: minor editing, mov->MP4, removed unneeded media prefix Created 6 years, 5 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/filters/ffmpeg_demuxer.cc ('k') | media/filters/pipeline_integration_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/ffmpeg_demuxer_unittest.cc
diff --git a/media/filters/ffmpeg_demuxer_unittest.cc b/media/filters/ffmpeg_demuxer_unittest.cc
index 638aa0c4fa51e8d5a65fc68e52a21da94398da73..9059db2bd1b4dbc5f8a5045b0cca284b9215fea2 100644
--- a/media/filters/ffmpeg_demuxer_unittest.cc
+++ b/media/filters/ffmpeg_demuxer_unittest.cc
@@ -916,6 +916,46 @@ TEST_F(FFmpegDemuxerTest, IsValidAnnexB) {
}
}
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_0) {
+ const char* file = "bear_rotate_0.mp4";
+ CreateDemuxer(file);
scherkus (not reviewing) 2014/07/09 23:01:27 any reason why you don't simply inline the "bear_r
+ InitializeDemuxer();
+
+ DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+ ASSERT_TRUE(stream);
+ ASSERT_EQ(VIDEO_ROTATION_0, stream->video_rotation());
+}
+
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_90) {
+ const char* file = "bear_rotate_90.mp4";
+ CreateDemuxer(file);
+ InitializeDemuxer();
+
+ DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+ ASSERT_TRUE(stream);
+ ASSERT_EQ(VIDEO_ROTATION_90, stream->video_rotation());
+}
+
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_180) {
+ const char* file = "bear_rotate_180.mp4";
+ CreateDemuxer(file);
+ InitializeDemuxer();
+
+ DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+ ASSERT_TRUE(stream);
+ ASSERT_EQ(VIDEO_ROTATION_180, stream->video_rotation());
+}
+
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_270) {
+ const char* file = "bear_rotate_270.mp4";
+ CreateDemuxer(file);
+ InitializeDemuxer();
+
+ DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+ ASSERT_TRUE(stream);
+ ASSERT_EQ(VIDEO_ROTATION_270, stream->video_rotation());
+}
+
#endif
} // namespace media
« no previous file with comments | « media/filters/ffmpeg_demuxer.cc ('k') | media/filters/pipeline_integration_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698