From 4f3a2eba6b989f28045be92615bd3fb214a359cc Mon Sep 17 00:00:00 2001
From: Artem Titov <titovartem@google.com>
Date: Tue, 26 Jan 2021 09:58:21 +0100
Subject: [PATCH] Destroy previous offer instead of leaking it in
 PeerConnectionInterfaceTest.ExtmapAllowMixedIsConfigurable

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}
---
 pc/peer_connection_interface_unittest.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pc/peer_connection_interface_unittest.cc b/pc/peer_connection_interface_unittest.cc
index 505fe6b9b1..e3d0ab61a3 100644
--- a/pc/peer_connection_interface_unittest.cc
+++ b/pc/peer_connection_interface_unittest.cc
@@ -3908,7 +3908,7 @@ TEST_P(PeerConnectionInterfaceTest, ExtmapAllowMixedIsConfigurable) {
   // Possible to set to false.
   config.offer_extmap_allow_mixed = false;
   CreatePeerConnection(config);
-  offer.release();
+  offer = nullptr;
   ASSERT_TRUE(DoCreateOffer(&offer, nullptr));
   EXPECT_FALSE(offer->description()->extmap_allow_mixed());
 }