Media Remoting: Add RemotingController.

media/base/pipeline_metadata.h

 // Copyright 2016 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_PIPELINE_METADATA_H_
 #define MEDIA_BASE_PIPELINE_METADATA_H_
 
 #include "base/time/time.h"
+#include "media/base/audio_decoder_config.h"
+#include "media/base/video_decoder_config.h"
 #include "media/base/video_rotation.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace media {
 
 // Metadata describing a pipeline once it has been initialized.
-struct PipelineMetadata {
-  PipelineMetadata()
-      : has_audio(false), has_video(false), video_rotation(VIDEO_ROTATION_0) {}
+struct MEDIA_EXPORT PipelineMetadata {
+  PipelineMetadata();
+  ~PipelineMetadata();
+
+  // Allow copy construction and assignment.
+  PipelineMetadata(const PipelineMetadata&);
+  PipelineMetadata& operator=(const PipelineMetadata&);

[xhwang, 2016/10/01 07:12:15]

hmm, do you need these?

[xjz, 2016/10/03 22:31:08]

These are required by compiler.

[xhwang, 2016/10/04 06:30:29]

OOC, won't the compiler auto-generate these if you don't have them here
explicitly? I might be missing something. So it seems we need more comments
here...

[xjz, 2016/10/04 19:21:28]

Removed the assignment constructor, but copy constructor is required by chromium
style. Added comment.

 
   bool has_audio;
   bool has_video;
+  AudioDecoderConfig audio_decoder_config;
+  VideoDecoderConfig video_decoder_config;
   gfx::Size natural_size;
   VideoRotation video_rotation;
   base::Time timeline_offset;
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_PIPELINE_METADATA_H_