admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Destroy previous offer instead of leaking it in PeerConnectionInterfaceTest.ExtmapAllowMixedIsConfigurable

Browse Source 
Bug: None
Change-Id: I41626e1ff3b13734509e83ead13eed5d85828503
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/203884
Reviewed-by: Per Kjellander <perkj@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#33071}
This commit is contained in:
Artem Titov 2021-01-26 09:58:21 +01:00 committed by Commit Bot
parent 0a03ed8492
commit 4f3a2eba6b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pc/peer_connection_interface_unittest.cc
View File

@ -3908,7 +3908,7 @@ TEST_P(PeerConnectionInterfaceTest, ExtmapAllowMixedIsConfigurable) {
// Possible to set to false. // Possible to set to false.
config.offer_extmap_allow_mixed = false; config.offer_extmap_allow_mixed = false;
CreatePeerConnection(config); CreatePeerConnection(config);
offer.release(); offer = nullptr;
ASSERT_TRUE(DoCreateOffer(&offer, nullptr)); ASSERT_TRUE(DoCreateOffer(&offer, nullptr));
EXPECT_FALSE(offer->description()->extmap_allow_mixed()); EXPECT_FALSE(offer->description()->extmap_allow_mixed());
} }