Reverting to previous behavior due to a downstream test expectation.
This reverts the change in behavior for setRemoteDescription,
introduced here:
https://webrtc-review.googlesource.com/c/src/+/206063
And disables the associated test.
Bug: webrtc:9987
Change-Id: I39a5664032a967a0a9cd336fa585d4d3880c88c5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/207162
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Tommi <tommi@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33248}
diff --git a/pc/sdp_offer_answer.cc b/pc/sdp_offer_answer.cc
index b0a594b..4317bca 100644
--- a/pc/sdp_offer_answer.cc
+++ b/pc/sdp_offer_answer.cc
@@ -4478,8 +4478,10 @@
const cricket::Candidate& c = candidate->candidate();
RTCError error = cricket::VerifyCandidate(c);
- if (!error.ok())
- return false;
+ if (!error.ok()) {
+ RTC_LOG(LS_WARNING) << "Invalid candidate: " << c.ToString();
+ return true;
+ }
pc_->AddRemoteCandidate(result.value()->name, c);
