Cast:Transport: Dividing A/V Initialization pipeline

media/cast/transport/cast_transport_config.cc

Index: media/cast/transport/cast_transport_config.cc
diff --git a/media/cast/transport/cast_transport_config.cc b/media/cast/transport/cast_transport_config.cc
index 1ec6be5485438ccd857188edd8417c0742306df7..5b8fbbe9f170b96d089abb223c109d594c2b88cc 100644
--- a/media/cast/transport/cast_transport_config.cc
+++ b/media/cast/transport/cast_transport_config.cc
@@ -18,17 +18,19 @@ RtpConfig::RtpConfig()
       max_delay_ms(kDefaultRtpMaxDelayMs),
       payload_type(0) {}
 
-CastTransportConfig::CastTransportConfig()
-    : audio_ssrc(0),
-      video_ssrc(0),
-      video_codec(kVp8),
-      audio_codec(kOpus),
-      audio_frequency(0),
-      audio_channels(0),
-      aes_key(""),
-      aes_iv_mask("") {}
-
-CastTransportConfig::~CastTransportConfig() {}
+CastTransportBaseConfig::CastTransportBaseConfig()
+    : ssrc(0), aes_key(""), aes_iv_mask("") {}

[Ami GONE FROM CHROMIUM, 2014/02/25 00:22:17]

std::string's default-initialize to empty strings; you don't need to explicitly
construct them here.

[mikhal1, 2014/02/25 18:07:02]

Done.

+
+CastTransportBaseConfig::~CastTransportBaseConfig() {}
+
+CastTransportAudioConfig::CastTransportAudioConfig()
+    : codec(kOpus), frequency(0), channels(0) {}
+
+CastTransportAudioConfig::~CastTransportAudioConfig() {}
+
+CastTransportVideoConfig::CastTransportVideoConfig() : codec(kVp8) {}
+
+CastTransportVideoConfig::~CastTransportVideoConfig() {}
 
 EncodedVideoFrame::EncodedVideoFrame()
     : codec(kVp8),
@@ -70,8 +72,7 @@ RtcpDlrrReportBlock::RtcpDlrrReportBlock()
 RtcpDlrrReportBlock::~RtcpDlrrReportBlock() {}
 
 SendRtcpFromRtpSenderData::SendRtcpFromRtpSenderData()
-    : packet_type_flags(0),
-      sending_ssrc(0) {}
+    : packet_type_flags(0), sending_ssrc(0) {}
 SendRtcpFromRtpSenderData::~SendRtcpFromRtpSenderData() {}
 
 }  // namespace transport