Split out dscp option from VideoOptions to new struct MediaChannelOptions.

talk/media/webrtc/webrtcvideoengine2.cc

Index: talk/media/webrtc/webrtcvideoengine2.cc
diff --git a/talk/media/webrtc/webrtcvideoengine2.cc b/talk/media/webrtc/webrtcvideoengine2.cc
index 2d5ec5383b3f30ff6ae6e2bb7c9b8ddfd655b102..403ff16420e646ad27538077e5023c485547521c 100644
--- a/talk/media/webrtc/webrtcvideoengine2.cc
+++ b/talk/media/webrtc/webrtcvideoengine2.cc
@@ -520,11 +520,13 @@ void WebRtcVideoEngine2::Init() {
 
 WebRtcVideoChannel2* WebRtcVideoEngine2::CreateChannel(
     webrtc::Call* call,
-    const VideoOptions& options) {
+    const MediaChannelOptions& options,
+    const VideoOptions& video_options) {
   RTC_DCHECK(initialized_);
-  LOG(LS_INFO) << "CreateChannel. Options: " << options.ToString();
-  return new WebRtcVideoChannel2(call, options, video_codecs_,
-      external_encoder_factory_, external_decoder_factory_);
+  LOG(LS_INFO) << "CreateChannel. Options: " << video_options.ToString();
+  return new WebRtcVideoChannel2(call, options, video_options, video_codecs_,
+                                 external_encoder_factory_,
+                                 external_decoder_factory_);
 }
 
 const std::vector<VideoCodec>& WebRtcVideoEngine2::codecs() const {
@@ -639,17 +641,19 @@ std::vector<VideoCodec> WebRtcVideoEngine2::GetSupportedCodecs() const {
 
 WebRtcVideoChannel2::WebRtcVideoChannel2(
     webrtc::Call* call,
-    const VideoOptions& options,
+    const MediaChannelOptions& options,
+    const VideoOptions& video_options,
     const std::vector<VideoCodec>& recv_codecs,
     WebRtcVideoEncoderFactory* external_encoder_factory,
     WebRtcVideoDecoderFactory* external_decoder_factory)
-    : call_(call),
+    : VideoMediaChannel(options),
+      call_(call),
       unsignalled_ssrc_handler_(&default_unsignalled_ssrc_handler_),
       external_encoder_factory_(external_encoder_factory),
       external_decoder_factory_(external_decoder_factory) {
   RTC_DCHECK(thread_checker_.CalledOnValidThread());
   SetDefaultOptions();
-  options_.SetAll(options);
+  options_.SetAll(video_options);
   if (options_.cpu_overuse_detection)
     signal_cpu_adaptation_ = *options_.cpu_overuse_detection;
   rtcp_receiver_report_ssrc_ = kDefaultRtcpReceiverReportSsrc;
@@ -661,7 +665,6 @@ WebRtcVideoChannel2::WebRtcVideoChannel2(
 
 void WebRtcVideoChannel2::SetDefaultOptions() {
   options_.cpu_overuse_detection = rtc::Optional<bool>(true);
-  options_.dscp = rtc::Optional<bool>(false);
   options_.suspend_below_min_bitrate = rtc::Optional<bool>(false);
   options_.screencast_min_bitrate_kbps = rtc::Optional<int>(0);
 }
@@ -794,6 +797,10 @@ bool WebRtcVideoChannel2::GetChangedSendParameters(
   return true;
 }
 
+rtc::DiffServCodePoint WebRtcVideoChannel2::MediaTypeDscpValue() const {
+  return rtc::DSCP_AF41;
+}
+
 bool WebRtcVideoChannel2::SetSendParameters(const VideoSendParameters& params) {
   TRACE_EVENT0("webrtc", "WebRtcVideoChannel2::SetSendParameters");
   LOG(LS_INFO) << "SetSendParameters: " << params.ToString();
@@ -846,9 +853,6 @@ bool WebRtcVideoChannel2::SetSendParameters(const VideoSendParameters& params) {
         signal_cpu_adaptation_ = *options_.cpu_overuse_detection;
       }
     }
-    rtc::DiffServCodePoint dscp =
-        options_.dscp.value_or(false) ? rtc::DSCP_AF41 : rtc::DSCP_DEFAULT;
-    MediaChannel::SetDscp(dscp);
   }
 
   {