Cleaned up sdp management (#1303)

lynckia · Sep 21, 2018 · 7474983 · 7474983
1 parent ca97119
commit 7474983
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 21 deletions.
diff --git a/erizo/src/erizo/MediaStream.cpp b/erizo/src/erizo/MediaStream.cpp
@@ -208,11 +208,6 @@ bool MediaStream::setRemoteSdp(std::shared_ptr<SdpInfo> sdp) {
   return true;
 }
 
-bool MediaStream::setLocalSdp(std::shared_ptr<SdpInfo> sdp) {
-  local_sdp_ = std::move(sdp);
-  return true;
-}
-
 void MediaStream::initializeStats() {
   log_stats_->getNode().insertStat("streamId", StringStat{getId()});
   log_stats_->getNode().insertStat("audioBitrate", CumulativeStat{0});

diff --git a/erizo/src/erizo/MediaStream.h b/erizo/src/erizo/MediaStream.h
@@ -75,7 +75,6 @@ class MediaStream: public MediaSink, public MediaSource, public FeedbackSink,
   void setMaxVideoBW(uint32_t max_video_bw);
   void syncClose();
   bool setRemoteSdp(std::shared_ptr<SdpInfo> sdp);
-  bool setLocalSdp(std::shared_ptr<SdpInfo> sdp);
 
   /**
    * Sends a PLI Packet
@@ -135,7 +134,7 @@ class MediaStream: public MediaSink, public MediaSource, public FeedbackSink,
   bool isAudioMuted() { return audio_muted_; }
   bool isVideoMuted() { return video_muted_; }
 
-  SdpInfo* getRemoteSdpInfo() { return remote_sdp_.get(); }
+  std::shared_ptr<SdpInfo> getRemoteSdpInfo() { return remote_sdp_; }
 
   virtual bool isSlideShowModeEnabled() { return slide_show_mode_; }
 
@@ -145,8 +144,8 @@ class MediaStream: public MediaSink, public MediaSource, public FeedbackSink,
   RtpExtensionProcessor& getRtpExtensionProcessor() { return connection_->getRtpExtensionProcessor(); }
   std::shared_ptr<Worker> getWorker() { return worker_; }
 
-  std::string& getId() { return stream_id_; }
-  std::string& getLabel() { return mslabel_; }
+  std::string getId() { return stream_id_; }
+  std::string getLabel() { return mslabel_; }
 
   bool isSourceSSRC(uint32_t ssrc);
   bool isSinkSSRC(uint32_t ssrc);
@@ -214,7 +213,6 @@ class MediaStream: public MediaSink, public MediaSource, public FeedbackSink,
   std::atomic<uint64_t> bitrate_from_max_quality_layer_;
  protected:
   std::shared_ptr<SdpInfo> remote_sdp_;
-  std::shared_ptr<SdpInfo> local_sdp_;
 };
 
 class PacketReader : public InboundHandler {

diff --git a/erizo/src/erizo/WebRtcConnection.cpp b/erizo/src/erizo/WebRtcConnection.cpp
@@ -152,12 +152,6 @@ bool WebRtcConnection::createOffer(bool video_enabled, bool audioEnabled, bool b
   std::string msg = this->getLocalSdp();
   maybeNotifyWebRtcConnectionEvent(global_state_, msg);
 
-  std::weak_ptr<WebRtcConnection> weak_this = shared_from_this();
-  forEachMediaStreamAsync([weak_this] (const std::shared_ptr<MediaStream> &media_stream) {
-    if (auto connection = weak_this.lock()) {
-      media_stream->setLocalSdp(connection->local_sdp_);
-    }
-  });
   return true;
 }
 
@@ -293,7 +287,6 @@ void WebRtcConnection::setRemoteSdpsToMediaStreams(std::string stream_id) {
     (*stream)->asyncTask([weak_this, stream_id] (const std::shared_ptr<MediaStream> &media_stream) {
       if (auto connection = weak_this.lock()) {
         media_stream->setRemoteSdp(connection->remote_sdp_);
-        media_stream->setLocalSdp(connection->local_sdp_);
         ELOG_DEBUG("%s message: setting remote SDP to stream, stream: %s", connection->toLog(), media_stream->getId());
         connection->onRemoteSdpsSetToMediaStreams(stream_id);
       }

diff --git a/erizo/src/erizo/rtp/FecReceiverHandler.cpp b/erizo/src/erizo/rtp/FecReceiverHandler.cpp
@@ -32,9 +32,8 @@ void FecReceiverHandler::notifyUpdate() {
   if (!stream) {
     return;
   }
-  SdpInfo* remote_sdp = stream->getRemoteSdpInfo();
   bool is_slide_show_mode_active = stream->isSlideShowModeEnabled();
-  if (!remote_sdp->supportPayloadType(RED_90000_PT) || is_slide_show_mode_active) {
+  if (!stream->getRemoteSdpInfo()->supportPayloadType(RED_90000_PT) || is_slide_show_mode_active) {
     enable();
   } else {
     disable();

diff --git a/erizo/src/test/utils/Mocks.h b/erizo/src/test/utils/Mocks.h
@@ -103,7 +103,6 @@ class MockMediaStream: public MediaStream {
     const std::string& media_stream_id, const std::string& media_stream_label,
     std::vector<RtpMap> rtp_mappings, bool is_publisher = true) :
   MediaStream(worker, connection, media_stream_id, media_stream_label, is_publisher) {
-    local_sdp_ = std::make_shared<SdpInfo>(rtp_mappings);
     remote_sdp_ = std::make_shared<SdpInfo>(rtp_mappings);
   }
 

diff --git a/erizo_controller/erizoJS/models/SessionDescription.js b/erizo_controller/erizoJS/models/SessionDescription.js
@@ -224,7 +224,7 @@ class SessionDescription {
   }
 
   getStreamInfo(info, stream) {
-    const streamId = stream.id;
+    const streamId = stream.getId();
     let videoSsrcList = [];
     let simulcastVideoSsrcList;