diff options
| author | Byoungchan Lee <daniel.l@hpcnt.com> | 2022-05-30 23:59:55 +0900 |
|---|---|---|
| committer | WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-05-31 00:58:04 +0000 |
| commit | 7d4634cef76a1ac244d4b83faaf4c617bf236b71 (patch) | |
| tree | 3aa4f71786a6f7351544ab8378d442521ca8a516 /pc/peer_connection_bundle_unittest.cc | |
| parent | ad924e5578776c016f695b37f6ebe45b10624696 (diff) | |
| download | webrtc-7d4634cef76a1ac244d4b83faaf4c617bf236b71.tar.gz | |
Don't create PacketSocketFactory inside BasicPortAllocatorSession
This extends AlwaysValidPointer to avoid creating a unique_ptr inside it.
Bug: webrtc:13145
Change-Id: I73a4f18d0a7037b57f575b04b134e4f7eadceb79
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/263240
Reviewed-by: Jonas Oreland <jonaso@webrtc.org>
Commit-Queue: Daniel.L (Byoungchan) Lee <daniel.l@hpcnt.com>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#37048}
Diffstat (limited to 'pc/peer_connection_bundle_unittest.cc')
| -rw-r--r-- | pc/peer_connection_bundle_unittest.cc | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/pc/peer_connection_bundle_unittest.cc b/pc/peer_connection_bundle_unittest.cc index fed4930f43..ad3d118510 100644 --- a/pc/peer_connection_bundle_unittest.cc +++ b/pc/peer_connection_bundle_unittest.cc @@ -224,8 +224,9 @@ class PeerConnectionBundleBaseTest : public ::testing::Test { WrapperPtr CreatePeerConnection(const RTCConfiguration& config) { auto* fake_network = NewFakeNetwork(); - auto port_allocator = - std::make_unique<cricket::BasicPortAllocator>(fake_network); + auto port_allocator = std::make_unique<cricket::BasicPortAllocator>( + fake_network, + std::make_unique<rtc::BasicPacketSocketFactory>(vss_.get())); port_allocator->set_flags(cricket::PORTALLOCATOR_DISABLE_TCP | cricket::PORTALLOCATOR_DISABLE_RELAY); port_allocator->set_step_delay(cricket::kMinimumStepDelay); |