Update to Pipeline Metadata and Decoder Stream for Orientation Data

media/base/demuxer_stream.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_DEMUXER_STREAM_H_
 #define MEDIA_BASE_DEMUXER_STREAM_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "media/base/media_export.h"
+#include "media/base/video_rotation.h"
 
 namespace media {
 
 class AudioDecoderConfig;
 class DecoderBuffer;
 class VideoDecoderConfig;
 
 class MEDIA_EXPORT DemuxerStream {
  public:
   enum Type {
@@ skipping 52 matching lines @@
   // Whether or not this DemuxerStream allows midstream configuration changes.
   //
   // A DemuxerStream that returns 'true' to this may return the 'kConfigChange'
   // status from a Read() call. In this case the client is expected to be
   // capable of taking appropriate action to handle config changes. Otherwise
   // audio_decoder_config() and video_decoder_config()'s return values are
   // guaranteed to remain constant, and the client may make optimizations based
   // on this.
   virtual bool SupportsConfigChanges() = 0;
 
+  VideoRotation video_rotation() { return video_rotation_; }
+  void set_video_rotation(VideoRotation video_rotation);

[scherkus (not reviewing), 2014/07/07 22:41:11]

this class is a pure virtual interface and we should try to keep it that way

in other words, declare video_rotation as pure virtual and remove the
video_rotation_ member variable and update implementations (most can return
VIDEO_ROTATION_0)

+
  protected:
   // Only allow concrete implementations to get deleted.
   virtual ~DemuxerStream();
+
+ private:
+  VideoRotation video_rotation_;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DEMUXER_STREAM_H_