Update to Pipeline Metadata and Decoder Stream for Orientation Data

media/filters/ffmpeg_demuxer_unittest.cc

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..0b14174f97eea425dcd7d19c20fa861dbe010efc 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.mov";

[scherkus (not reviewing), 2014/07/09 22:15:16]

* goes with the type

GOOD: const char* foo
BAD: const char * foo
BAD: const char *foo

+  CreateDemuxer(file);
+  InitializeDemuxer();
+
+  DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+  ASSERT_TRUE(stream);
+  ASSERT_TRUE(stream->video_rotation() == media::VIDEO_ROTATION_0);

[scherkus (not reviewing), 2014/07/09 22:15:16]

gtest's {EXPECT,ASSERT}_EQ() uses (expected, actual) ordering of arguments

also drop media:: prefixes inside media namespace

in other words:

ASSERT_EQ(VIDEO_ROTATION_0, stream->video_rotation());

+}
+
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_90) {
+  const char * file = "bear_rotate_90.mov";

[scherkus (not reviewing), 2014/07/09 22:15:16]

ditto

+  CreateDemuxer(file);
+  InitializeDemuxer();
+
+  DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+  ASSERT_TRUE(stream);
+  ASSERT_TRUE(stream->video_rotation() == media::VIDEO_ROTATION_90);

[scherkus (not reviewing), 2014/07/09 22:15:16]

ditto

+}
+
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_180) {
+  const char * file = "bear_rotate_180.mov";

[scherkus (not reviewing), 2014/07/09 22:15:16]

ditto

+  CreateDemuxer(file);
+  InitializeDemuxer();
+
+  DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+  ASSERT_TRUE(stream);
+  ASSERT_TRUE(stream->video_rotation() == media::VIDEO_ROTATION_180);

[scherkus (not reviewing), 2014/07/09 22:15:16]

ditto

+}
+
+TEST_F(FFmpegDemuxerTest, Rotate_Metadata_270) {
+  const char * file = "bear_rotate_270.mov";

[scherkus (not reviewing), 2014/07/09 22:15:16]

ditto

+  CreateDemuxer(file);
+  InitializeDemuxer();
+
+  DemuxerStream* stream = demuxer_->GetStream(DemuxerStream::VIDEO);
+  ASSERT_TRUE(stream);
+  ASSERT_TRUE(stream->video_rotation() == media::VIDEO_ROTATION_270);

[scherkus (not reviewing), 2014/07/09 22:15:16]

ditto

+}
+
 #endif
 
 }  // namespace media