Reland of: Separating SCTP code from BaseChannel/MediaChannel.

webrtc/pc/channel.cc

 /*
  *  Copyright 2004 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 51 matching lines @@
 }  // namespace
 
 enum {
   MSG_EARLYMEDIATIMEOUT = 1,
   MSG_SEND_RTP_PACKET,
   MSG_SEND_RTCP_PACKET,
   MSG_CHANNEL_ERROR,
   MSG_READYTOSENDDATA,
   MSG_DATARECEIVED,
   MSG_FIRSTPACKETRECEIVED,
-  MSG_STREAMCLOSEDREMOTELY,
 };
 
 // Value specified in RFC 5764.
 static const char kDtlsSrtpExporterLabel[] = "EXTRACTOR-dtls_srtp";
 
 static const int kAgcMinus10db = -10;
 
 static void SafeSetError(const std::string& message, std::string* error_desc) {
   if (error_desc) {
     *error_desc = message;
@@ skipping 841 matching lines @@
         << selected_candidate_pair_->local_candidate().ToSensitiveString()
         << "->"
         << selected_candidate_pair_->remote_candidate().ToSensitiveString();
 
   was_ever_writable_ = true;
   MaybeSetupDtlsSrtp_n();
   writable_ = true;
   UpdateMediaSendRecvState();
 }
 
-void BaseChannel::SignalDtlsSetupFailure_n(bool rtcp) {
+void BaseChannel::SignalDtlsSrtpSetupFailure_n(bool rtcp) {
   RTC_DCHECK(network_thread_->IsCurrent());
   invoker_.AsyncInvoke<void>(
       RTC_FROM_HERE, signaling_thread(),
-      Bind(&BaseChannel::SignalDtlsSetupFailure_s, this, rtcp));
+      Bind(&BaseChannel::SignalDtlsSrtpSetupFailure_s, this, rtcp));
 }
 
-void BaseChannel::SignalDtlsSetupFailure_s(bool rtcp) {
+void BaseChannel::SignalDtlsSrtpSetupFailure_s(bool rtcp) {
   RTC_DCHECK(signaling_thread() == rtc::Thread::Current());
-  SignalDtlsSetupFailure(this, rtcp);
+  SignalDtlsSrtpSetupFailure(this, rtcp);
 }
 
 bool BaseChannel::SetDtlsSrtpCryptoSuites_n(TransportChannel* tc, bool rtcp) {
   std::vector<int> crypto_suites;
   // We always use the default SRTP crypto suites for RTCP, but we may use
   // different crypto suites for RTP depending on the media type.
   if (!rtcp) {
     GetSrtpCryptoSuites_n(&crypto_suites);
   } else {
     GetDefaultSrtpCryptoSuites(crypto_options(), &crypto_suites);
@@ skipping 100 matching lines @@
 void BaseChannel::MaybeSetupDtlsSrtp_n() {
   if (srtp_filter_.IsActive()) {
     return;
   }
 
   if (!ShouldSetupDtlsSrtp_n()) {
     return;
   }
 
   if (!SetupDtlsSrtp_n(false)) {
-    SignalDtlsSetupFailure_n(false);
+    SignalDtlsSrtpSetupFailure_n(false);
     return;
   }
 
   if (rtcp_transport_channel_) {
     if (!SetupDtlsSrtp_n(true)) {
-      SignalDtlsSetupFailure_n(true);
+      SignalDtlsSrtpSetupFailure_n(true);
       return;
     }
   }
 }
 
 void BaseChannel::ChannelNotWritable_n() {
   RTC_DCHECK(network_thread_->IsCurrent());
   if (!writable_)
     return;
 
@@ skipping 1059 matching lines @@
     VideoMediaChannel* media_channel, const VideoMediaInfo &info) {
   RTC_DCHECK(media_channel == this->media_channel());
   SignalMediaMonitor(this, info);
 }
 
 void VideoChannel::GetSrtpCryptoSuites_n(
     std::vector<int>* crypto_suites) const {
   GetSupportedVideoCryptoSuites(crypto_options(), crypto_suites);
 }
 
-DataChannel::DataChannel(rtc::Thread* worker_thread,
-                         rtc::Thread* network_thread,
-                         DataMediaChannel* media_channel,
-                         TransportController* transport_controller,
-                         const std::string& content_name,
-                         bool rtcp,
-                         bool srtp_required)
+RtpDataChannel::RtpDataChannel(rtc::Thread* worker_thread,
+                               rtc::Thread* network_thread,
+                               DataMediaChannel* media_channel,
+                               TransportController* transport_controller,
+                               const std::string& content_name,
+                               bool rtcp,
+                               bool srtp_required)
     : BaseChannel(worker_thread,
                   network_thread,
                   media_channel,
                   transport_controller,
                   content_name,
                   rtcp,
-                  srtp_required),
-      data_channel_type_(cricket::DCT_NONE),
-      ready_to_send_data_(false) {}
+                  srtp_required) {}
 
-DataChannel::~DataChannel() {
-  TRACE_EVENT0("webrtc", "DataChannel::~DataChannel");
+RtpDataChannel::~RtpDataChannel() {
+  TRACE_EVENT0("webrtc", "RtpDataChannel::~RtpDataChannel");
   StopMediaMonitor();
   // this can't be done in the base class, since it calls a virtual
   DisableMedia_w();
 
   Deinit();
 }
 
-bool DataChannel::Init_w(const std::string* bundle_transport_name) {
+bool RtpDataChannel::Init_w(const std::string* bundle_transport_name) {
   if (!BaseChannel::Init_w(bundle_transport_name)) {
     return false;
   }
-  media_channel()->SignalDataReceived.connect(
-      this, &DataChannel::OnDataReceived);
+  media_channel()->SignalDataReceived.connect(this,
+                                              &RtpDataChannel::OnDataReceived);
   media_channel()->SignalReadyToSend.connect(
-      this, &DataChannel::OnDataChannelReadyToSend);
-  media_channel()->SignalStreamClosedRemotely.connect(
-      this, &DataChannel::OnStreamClosedRemotely);
+      this, &RtpDataChannel::OnDataChannelReadyToSend);
   return true;
 }
 
-bool DataChannel::SendData(const SendDataParams& params,
-                           const rtc::CopyOnWriteBuffer& payload,
-                           SendDataResult* result) {
+bool RtpDataChannel::SendData(const SendDataParams& params,
+                              const rtc::CopyOnWriteBuffer& payload,
+                              SendDataResult* result) {
   return InvokeOnWorker(
       RTC_FROM_HERE, Bind(&DataMediaChannel::SendData, media_channel(), params,
                           payload, result));
 }
 
-const ContentInfo* DataChannel::GetFirstContent(
+const ContentInfo* RtpDataChannel::GetFirstContent(
     const SessionDescription* sdesc) {
   return GetFirstDataContent(sdesc);
 }
 
-bool DataChannel::WantsPacket(bool rtcp, const rtc::CopyOnWriteBuffer* packet) {
-  if (data_channel_type_ == DCT_SCTP) {
-    // TODO(pthatcher): Do this in a more robust way by checking for
-    // SCTP or DTLS.
-    return !IsRtpPacket(packet->data(), packet->size());
-  } else if (data_channel_type_ == DCT_RTP) {
-    return BaseChannel::WantsPacket(rtcp, packet);
-  }
-  return false;
-}
-
-bool DataChannel::SetDataChannelType(DataChannelType new_data_channel_type,
-                                     std::string* error_desc) {
-  // It hasn't been set before, so set it now.
-  if (data_channel_type_ == DCT_NONE) {
-    data_channel_type_ = new_data_channel_type;
-    return true;
-  }
-
-  // It's been set before, but doesn't match.  That's bad.
-  if (data_channel_type_ != new_data_channel_type) {
-    std::ostringstream desc;
-    desc << "Data channel type mismatch."
-         << " Expected " << data_channel_type_
-         << " Got " << new_data_channel_type;
-    SafeSetError(desc.str(), error_desc);
-    return false;
-  }
-
-  // It's hasn't changed.  Nothing to do.
-  return true;
-}
-
-bool DataChannel::SetDataChannelTypeFromContent(
+bool RtpDataChannel::CheckDataChannelTypeFromContent(
     const DataContentDescription* content,
     std::string* error_desc) {
   bool is_sctp = ((content->protocol() == kMediaProtocolSctp) ||
                   (content->protocol() == kMediaProtocolDtlsSctp));
-  DataChannelType data_channel_type = is_sctp ? DCT_SCTP : DCT_RTP;
-  return SetDataChannelType(data_channel_type, error_desc);
+  // It's been set before, but doesn't match.  That's bad.
+  if (is_sctp) {
+    SafeSetError("Data channel type mismatch. Expected RTP, got SCTP.",
+                 error_desc);
+    return false;
+  }
+  return true;
 }
 
-bool DataChannel::SetLocalContent_w(const MediaContentDescription* content,
-                                    ContentAction action,
-                                    std::string* error_desc) {
-  TRACE_EVENT0("webrtc", "DataChannel::SetLocalContent_w");
+bool RtpDataChannel::SetLocalContent_w(const MediaContentDescription* content,
+                                       ContentAction action,
+                                       std::string* error_desc) {
+  TRACE_EVENT0("webrtc", "RtpDataChannel::SetLocalContent_w");
   RTC_DCHECK(worker_thread() == rtc::Thread::Current());
   LOG(LS_INFO) << "Setting local data description";
 
   const DataContentDescription* data =
       static_cast<const DataContentDescription*>(content);
   RTC_DCHECK(data != NULL);
   if (!data) {
     SafeSetError("Can't find data content in local description.", error_desc);
     return false;
   }
 
-  if (!SetDataChannelTypeFromContent(data, error_desc)) {
+  if (!CheckDataChannelTypeFromContent(data, error_desc)) {
     return false;
   }
 
-  if (data_channel_type_ == DCT_RTP) {
-    if (!SetRtpTransportParameters(content, action, CS_LOCAL, error_desc)) {
-      return false;
-    }
+  if (!SetRtpTransportParameters(content, action, CS_LOCAL, error_desc)) {
+    return false;
   }
 
-  // FYI: We send the SCTP port number (not to be confused with the
-  // underlying UDP port number) as a codec parameter.  So even SCTP
-  // data channels need codecs.
   DataRecvParameters recv_params = last_recv_params_;
   RtpParametersFromMediaDescription(data, &recv_params);
   if (!media_channel()->SetRecvParameters(recv_params)) {
     SafeSetError("Failed to set remote data description recv parameters.",
                  error_desc);
     return false;
   }
-  if (data_channel_type_ == DCT_RTP) {
-    for (const DataCodec& codec : data->codecs()) {
-      bundle_filter()->AddPayloadType(codec.id);
-    }
+  for (const DataCodec& codec : data->codecs()) {
+    bundle_filter()->AddPayloadType(codec.id);
   }
   last_recv_params_ = recv_params;
 
   // TODO(pthatcher): Move local streams into DataSendParameters, and
   // only give it to the media channel once we have a remote
   // description too (without a remote description, we won't be able
   // to send them anyway).
   if (!UpdateLocalStreams_w(data->streams(), action, error_desc)) {
     SafeSetError("Failed to set local data description streams.", error_desc);
     return false;
   }
 
   set_local_content_direction(content->direction());
   UpdateMediaSendRecvState_w();
   return true;
 }
 
-bool DataChannel::SetRemoteContent_w(const MediaContentDescription* content,
-                                     ContentAction action,
-                                     std::string* error_desc) {
-  TRACE_EVENT0("webrtc", "DataChannel::SetRemoteContent_w");
+bool RtpDataChannel::SetRemoteContent_w(const MediaContentDescription* content,
+                                        ContentAction action,
+                                        std::string* error_desc) {
+  TRACE_EVENT0("webrtc", "RtpDataChannel::SetRemoteContent_w");
   RTC_DCHECK(worker_thread() == rtc::Thread::Current());
 
   const DataContentDescription* data =
       static_cast<const DataContentDescription*>(content);
   RTC_DCHECK(data != NULL);
   if (!data) {
     SafeSetError("Can't find data content in remote description.", error_desc);
     return false;
   }
 
   // If the remote data doesn't have codecs and isn't an update, it
   // must be empty, so ignore it.
   if (!data->has_codecs() && action != CA_UPDATE) {
     return true;
   }
 
-  if (!SetDataChannelTypeFromContent(data, error_desc)) {
+  if (!CheckDataChannelTypeFromContent(data, error_desc)) {
     return false;
   }
 
   LOG(LS_INFO) << "Setting remote data description";
-  if (data_channel_type_ == DCT_RTP &&
-      !SetRtpTransportParameters(content, action, CS_REMOTE, error_desc)) {
+  if (!SetRtpTransportParameters(content, action, CS_REMOTE, error_desc)) {
     return false;
   }
 
-
   DataSendParameters send_params = last_send_params_;
   RtpSendParametersFromMediaDescription<DataCodec>(data, &send_params);
   if (!media_channel()->SetSendParameters(send_params)) {
     SafeSetError("Failed to set remote data description send parameters.",
                  error_desc);
     return false;
   }
   last_send_params_ = send_params;
 
   // TODO(pthatcher): Move remote streams into DataRecvParameters,
   // and only give it to the media channel once we have a local
   // description too (without a local description, we won't be able to
   // recv them anyway).
   if (!UpdateRemoteStreams_w(data->streams(), action, error_desc)) {
     SafeSetError("Failed to set remote data description streams.",
                  error_desc);
     return false;
   }
 
   set_remote_content_direction(content->direction());
   UpdateMediaSendRecvState_w();
   return true;
 }
 
-void DataChannel::UpdateMediaSendRecvState_w() {
+void RtpDataChannel::UpdateMediaSendRecvState_w() {
   // Render incoming data if we're the active call, and we have the local
   // content. We receive data on the default channel and multiplexed streams.
   bool recv = IsReadyToReceiveMedia_w();
   if (!media_channel()->SetReceive(recv)) {
     LOG(LS_ERROR) << "Failed to SetReceive on data channel";
   }
 
   // Send outgoing data if we're the active call, we have the remote content,
   // and we have had some form of connectivity.
   bool send = IsReadyToSendMedia_w();
   if (!media_channel()->SetSend(send)) {
     LOG(LS_ERROR) << "Failed to SetSend on data channel";
   }
 
   // Trigger SignalReadyToSendData asynchronously.
   OnDataChannelReadyToSend(send);
 
   LOG(LS_INFO) << "Changing data state, recv=" << recv << " send=" << send;
 }
 
-void DataChannel::OnMessage(rtc::Message *pmsg) {
+void RtpDataChannel::OnMessage(rtc::Message* pmsg) {
   switch (pmsg->message_id) {
     case MSG_READYTOSENDDATA: {
       DataChannelReadyToSendMessageData* data =
           static_cast<DataChannelReadyToSendMessageData*>(pmsg->pdata);
       ready_to_send_data_ = data->data();
       SignalReadyToSendData(ready_to_send_data_);
       delete data;
       break;
     }
     case MSG_DATARECEIVED: {
       DataReceivedMessageData* data =
           static_cast<DataReceivedMessageData*>(pmsg->pdata);
-      SignalDataReceived(this, data->params, data->payload);
+      SignalDataReceived(data->params, data->payload);
       delete data;
       break;
     }
     case MSG_CHANNEL_ERROR: {
       const DataChannelErrorMessageData* data =
           static_cast<DataChannelErrorMessageData*>(pmsg->pdata);
       delete data;
       break;
     }
-    case MSG_STREAMCLOSEDREMOTELY: {
-      rtc::TypedMessageData<uint32_t>* data =
-          static_cast<rtc::TypedMessageData<uint32_t>*>(pmsg->pdata);
-      SignalStreamClosedRemotely(data->data());
-      delete data;
-      break;
-    }
     default:
       BaseChannel::OnMessage(pmsg);
       break;
   }
 }
 
-void DataChannel::OnConnectionMonitorUpdate(
-    ConnectionMonitor* monitor, const std::vector<ConnectionInfo>& infos) {
+void RtpDataChannel::OnConnectionMonitorUpdate(
+    ConnectionMonitor* monitor,
+    const std::vector<ConnectionInfo>& infos) {
   SignalConnectionMonitor(this, infos);
 }
 
-void DataChannel::StartMediaMonitor(int cms) {
+void RtpDataChannel::StartMediaMonitor(int cms) {
   media_monitor_.reset(new DataMediaMonitor(media_channel(), worker_thread(),
       rtc::Thread::Current()));
-  media_monitor_->SignalUpdate.connect(
-      this, &DataChannel::OnMediaMonitorUpdate);
+  media_monitor_->SignalUpdate.connect(this,
+                                       &RtpDataChannel::OnMediaMonitorUpdate);
   media_monitor_->Start(cms);
 }
 
-void DataChannel::StopMediaMonitor() {
+void RtpDataChannel::StopMediaMonitor() {
   if (media_monitor_) {
     media_monitor_->Stop();
     media_monitor_->SignalUpdate.disconnect(this);
     media_monitor_.reset();
   }
 }
 
-void DataChannel::OnMediaMonitorUpdate(
-    DataMediaChannel* media_channel, const DataMediaInfo& info) {
+void RtpDataChannel::OnMediaMonitorUpdate(DataMediaChannel* media_channel,
+                                          const DataMediaInfo& info) {
   RTC_DCHECK(media_channel == this->media_channel());
   SignalMediaMonitor(this, info);
 }
 
-void DataChannel::OnDataReceived(
-    const ReceiveDataParams& params, const char* data, size_t len) {
+void RtpDataChannel::OnDataReceived(const ReceiveDataParams& params,
+                                    const char* data,
+                                    size_t len) {
   DataReceivedMessageData* msg = new DataReceivedMessageData(
       params, data, len);
   signaling_thread()->Post(RTC_FROM_HERE, this, MSG_DATARECEIVED, msg);
 }
 
-void DataChannel::OnDataChannelError(uint32_t ssrc,
-                                     DataMediaChannel::Error err) {
+void RtpDataChannel::OnDataChannelError(uint32_t ssrc,
+                                        DataMediaChannel::Error err) {
   DataChannelErrorMessageData* data = new DataChannelErrorMessageData(
       ssrc, err);
   signaling_thread()->Post(RTC_FROM_HERE, this, MSG_CHANNEL_ERROR, data);
 }
 
-void DataChannel::OnDataChannelReadyToSend(bool writable) {
+void RtpDataChannel::OnDataChannelReadyToSend(bool writable) {
   // This is usded for congestion control to indicate that the stream is ready
   // to send by the MediaChannel, as opposed to OnReadyToSend, which indicates
   // that the transport channel is ready.
   signaling_thread()->Post(RTC_FROM_HERE, this, MSG_READYTOSENDDATA,
                            new DataChannelReadyToSendMessageData(writable));
 }
 
-void DataChannel::GetSrtpCryptoSuites_n(std::vector<int>* crypto_suites) const {
+void RtpDataChannel::GetSrtpCryptoSuites_n(
+    std::vector<int>* crypto_suites) const {
   GetSupportedDataCryptoSuites(crypto_options(), crypto_suites);
 }
 
-bool DataChannel::ShouldSetupDtlsSrtp_n() const {
-  return data_channel_type_ == DCT_RTP && BaseChannel::ShouldSetupDtlsSrtp_n();
-}
-
-void DataChannel::OnStreamClosedRemotely(uint32_t sid) {
-  rtc::TypedMessageData<uint32_t>* message =
-      new rtc::TypedMessageData<uint32_t>(sid);
-  signaling_thread()->Post(RTC_FROM_HERE, this, MSG_STREAMCLOSEDREMOTELY,
-                           message);
-}
-
 }  // namespace cricket