From 91160efca4bbd361253f38bf1f2a502401647bd0 Mon Sep 17 00:00:00 2001 From: Tommi Date: Thu, 30 Mar 2023 01:09:45 +0200 Subject: [PATCH] Call HasUsedDataChannels() instead of HasDataChannels. ...when checking if negotiation is needed. I noticed this call site after we recently updated PeerConnection here: https://webrtc-review.googlesource.com/c/src/+/297860 Bug: chromium:1423562 Change-Id: Id37c938d731eadfccff44c95ef757a3cabd64936 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/299480 Commit-Queue: Tomas Gunnarsson Reviewed-by: Harald Alvestrand Cr-Commit-Position: refs/heads/main@{#39720} --- pc/sdp_offer_answer.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pc/sdp_offer_answer.cc b/pc/sdp_offer_answer.cc index f19cd13ac7..9c0e7eb860 100644 --- a/pc/sdp_offer_answer.cc +++ b/pc/sdp_offer_answer.cc @@ -3311,7 +3311,7 @@ bool SdpOfferAnswerHandler::CheckIfNegotiationIsNeeded() { // 4. If connection has created any RTCDataChannels, and no m= section in // description has been negotiated yet for data, return true. - if (data_channel_controller()->HasDataChannels()) { + if (data_channel_controller()->HasUsedDataChannels()) { if (!cricket::GetFirstDataContent(description->description()->contents())) return true; }