Update to Pipeline Metadata and Decoder Stream for Orientation Data

media/base/video_decoder_config.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_VIDEO_DECODER_CONFIG_H_
 #define MEDIA_BASE_VIDEO_DECODER_CONFIG_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 117 matching lines @@
   // Optional byte data required to initialize video decoders, such as H.264
   // AAVC data.
   const uint8* extra_data() const;
   size_t extra_data_size() const;
 
   // Whether the video stream is potentially encrypted.
   // Note that in a potentially encrypted video stream, individual buffers
   // 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_;
 
   VideoFrame::Format format_;
 
   gfx::Size coded_size_;
   gfx::Rect visible_rect_;
   gfx::Size natural_size_;
 
   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
   // typically small, the performance impact is minimal.
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_VIDEO_DECODER_CONFIG_H_