Update to Pipeline Metadata and Decoder Stream for Orientation Data

media/base/video_decoder_config.h

Index: media/base/video_decoder_config.h
diff --git a/media/base/video_decoder_config.h b/media/base/video_decoder_config.h
index 3cc33d927158a5c279c2e91b0e5dbd9aad6467e1..1cb6535e0ca30aea3d3ae5b004b228e4d1c324d1 100644
--- a/media/base/video_decoder_config.h
+++ b/media/base/video_decoder_config.h
@@ -135,6 +135,10 @@ class MEDIA_EXPORT VideoDecoderConfig {
   // can be encrypted or not encrypted.
   bool is_encrypted() const;
 
+  // Rotation of frame (0, 90, 180, 270) given in metadata
+  int rotation() const;

[scherkus (not reviewing), 2014/07/02 01:44:46]

xhwang: do you think rotation() should be part of a video decoder config? should
we perhaps considering sticking this on the DemuxerStream interface?

+  void set_rotation(int rotation);

[scherkus (not reviewing), 2014/07/02 01:44:46]

while FFmpeg may prefer to use an int, lets make it easier on downstream
consumers of this information by defining an enum that is a strict subset of
values we support

for example, something like:
enum VideoRotation {
  ROTATION_0,
  ROTATION_90,
  ROTATION_180,
  ROTATION_270,
};

+
  private:
   VideoCodec codec_;
   VideoCodecProfile profile_;
@@ -148,6 +152,7 @@ class MEDIA_EXPORT VideoDecoderConfig {
   std::vector<uint8> extra_data_;
 
   bool is_encrypted_;
+  int rotation_;
 
   // Not using DISALLOW_COPY_AND_ASSIGN here intentionally to allow the compiler
   // generated copy constructor and assignment operator. Since the extra data is