Fix direct dependencies in //media/cast/BUILD.gn.

media/cast/cast_config.h

 // Copyright 2013 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_CAST_CAST_CONFIG_H_
 #define MEDIA_CAST_CAST_CONFIG_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <string>
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/shared_memory.h"
 #include "base/single_thread_task_runner.h"
 #include "base/time/time.h"
-#include "media/cast/net/cast_transport_config.h"
+//#include "media/cast/net/cast_transport_config.h"
 
 namespace media {
 class VideoEncodeAccelerator;
 
 namespace cast {
 
+enum Codec {

[Wez, 2016/10/29 21:24:18]

As per crbug.com/530839, do these really belong here, or under an explicit
"defines" header under //media/cast/common?  (And should the common CastConfig
itself live there, too?)

[miu, 2016/10/31 02:10:34]

Not entirely sure. I wanted to plan all this out better in a project where we
refactor the code a bit to allow media/cast to be built as a standalone library.
So, I'm not sure if this would go in common/ ot a new public/ dir. TBD...

+  CODEC_UNKNOWN,
+  CODEC_AUDIO_OPUS,
+  CODEC_AUDIO_PCM16,
+  CODEC_AUDIO_AAC,
+  CODEC_AUDIO_REMOTE,
+  CODEC_VIDEO_FAKE,
+  CODEC_VIDEO_VP8,
+  CODEC_VIDEO_H264,
+  CODEC_VIDEO_REMOTE,
+  CODEC_LAST = CODEC_VIDEO_REMOTE
+};
+
+// Describes the content being transported over RTP streams.
+enum class RtpPayloadType {
+  UNKNOWN = -1,
+
+  // Cast Streaming will encode raw audio frames using one of its available
+  // codec implementations, and transport encoded data in the RTP stream.
+  FIRST = 96,
+  AUDIO_OPUS = 96,
+  AUDIO_AAC = 97,
+  AUDIO_PCM16 = 98,
+
+  // Audio frame data is not modified, and should be transported reliably and
+  // in-sequence. No assumptions about the data can be made.
+  REMOTE_AUDIO = 99,
+
+  AUDIO_LAST = REMOTE_AUDIO,
+
+  // Cast Streaming will encode raw video frames using one of its available
+  // codec implementations, and transport encoded data in the RTP stream.
+  VIDEO_VP8 = 100,
+  VIDEO_H264 = 101,
+
+  // Video frame data is not modified, and should be transported reliably and
+  // in-sequence. No assumptions about the data can be made.
+  REMOTE_VIDEO = 102,
+
+  LAST = REMOTE_VIDEO
+};
+
 // TODO(miu): Eliminate these after moving "default config" into the top-level
 // media/cast directory.  http://crbug.com/530839
 enum SuggestedDefaults {
   // Audio encoder bitrate.  Zero means "auto," which asks the encoder to select
   // a bitrate that dynamically adjusts to the content.  Otherwise, a constant
   // bitrate is used.
   kDefaultAudioEncoderBitrate = 0,
 
   // Suggested default audio sampling rate.
   kDefaultAudioSamplingRate = 48000,
@@ skipping 178 matching lines @@
 typedef base::Callback<void(std::unique_ptr<base::SharedMemory>)>
     ReceiveVideoEncodeMemoryCallback;
 typedef base::Callback<void(size_t size,
                             const ReceiveVideoEncodeMemoryCallback&)>
     CreateVideoEncodeMemoryCallback;
 
 }  // namespace cast
 }  // namespace media
 
 #endif  // MEDIA_CAST_CAST_CONFIG_H_