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..3ce9ba7790d4cfa92f1643a3dd04cac205a9ef80 100644
--- a/media/base/video_decoder_config.h
+++ b/media/base/video_decoder_config.h
@@ -66,6 +66,8 @@ enum VideoCodecProfile {
   VIDEO_CODEC_PROFILE_MAX = VP9PROFILE_MAX,
 };
 
+enum VideoRotation { kRotate0 = 0, kRotate90, kRotate180, kRotate270 };

[scherkus (not reviewing), 2014/07/07 20:52:38]

let's move this into its own file (media/base/video_rotation.h), add some
comments, and use MACRO_STYLE for the names as per
http://dev.chromium.org/developers/coding-style#Naming

we'll tack this onto the DemuxerStream interface as a video_rotation() getter
since rotation information isn't really part of a video decoder configuration
(i.e., video decoders don't make any use of the information)

+
 class MEDIA_EXPORT VideoDecoderConfig {
  public:
   // Constructs an uninitialized object. Clients should call Initialize() with
@@ -135,6 +137,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
+  VideoRotation rotation() const;
+  void set_rotation(VideoRotation rotation);
+
  private:
   VideoCodec codec_;
   VideoCodecProfile profile_;
@@ -148,6 +154,7 @@ class MEDIA_EXPORT VideoDecoderConfig {
   std::vector<uint8> extra_data_;
 
   bool is_encrypted_;
+  VideoRotation rotation_;
 
   // Not using DISALLOW_COPY_AND_ASSIGN here intentionally to allow the compiler
   // generated copy constructor and assignment operator. Since the extra data is