[Unified Plan] Support legacy endpoints that do not use a=mid
These legacy endpoints were supported with Plan B since the SDP
parser would fill in default MID values if a=mid was absent that
happened to match the default offered MIDs.
With Unified Plan, these default MIDs changed so the autofilled
MIDs do not match any more.
This CL adds information to the SessionDescription struct to
indicate whether or not a=mid was present and modified
PeerConnection::SetRemoteDescription to copy MIDs from the local
description if the a=mid lines are not present.
Bug: webrtc:9540
Change-Id: Ibf923b4ad59edb0facd06ddbd01cc10c62fc48e6
Reviewed-on: https://webrtc-review.googlesource.com/c/114820
Commit-Queue: Steve Anton <steveanton@webrtc.org>
Reviewed-by: Amit Hilbuch <amithi@webrtc.org>
Reviewed-by: Seth Hampson <shampson@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26054}diff --git a/pc/mediasession_unittest.cc b/pc/mediasession_unittest.cc
index fde824d..a2a216b 100644
--- a/pc/mediasession_unittest.cc
+++ b/pc/mediasession_unittest.cc
@@ -427,14 +427,14 @@
std::unique_ptr<SessionDescription> desc;
if (has_current_desc) {
current_desc = absl::make_unique<SessionDescription>();
- EXPECT_TRUE(current_desc->AddTransportInfo(TransportInfo(
+ current_desc->AddTransportInfo(TransportInfo(
"audio",
- TransportDescription(current_audio_ufrag, current_audio_pwd))));
- EXPECT_TRUE(current_desc->AddTransportInfo(TransportInfo(
+ TransportDescription(current_audio_ufrag, current_audio_pwd)));
+ current_desc->AddTransportInfo(TransportInfo(
"video",
- TransportDescription(current_video_ufrag, current_video_pwd))));
- EXPECT_TRUE(current_desc->AddTransportInfo(TransportInfo(
- "data", TransportDescription(current_data_ufrag, current_data_pwd))));
+ TransportDescription(current_video_ufrag, current_video_pwd)));
+ current_desc->AddTransportInfo(TransportInfo(
+ "data", TransportDescription(current_data_ufrag, current_data_pwd)));
}
if (offer) {
desc = f1_.CreateOffer(options, current_desc.get());