Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(325)

Unified Diff: net/quic/core/quic_multipath_received_packet_manager_test.cc

Issue 2417023003: Remove stl_util's deletion functions from: (Closed)
Patch Set: internal snapshot 10 Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/quic/core/quic_multipath_received_packet_manager.cc ('k') | net/quic/test_tools/crypto_test_utils.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_multipath_received_packet_manager_test.cc
diff --git a/net/quic/core/quic_multipath_received_packet_manager_test.cc b/net/quic/core/quic_multipath_received_packet_manager_test.cc
index 0dc2e2195d24f5190ddc0e2aa57119bb6ce156c1..effae4df069cd037ff97806c1334ec9bf1c78001 100644
--- a/net/quic/core/quic_multipath_received_packet_manager_test.cc
+++ b/net/quic/core/quic_multipath_received_packet_manager_test.cc
@@ -4,6 +4,7 @@
#include "net/quic/core/quic_multipath_received_packet_manager.h"
+#include "base/memory/ptr_util.h"
#include "net/quic/core/quic_connection_stats.h"
#include "net/quic/core/quic_flags.h"
#include "net/quic/test_tools/quic_test_utils.h"
@@ -28,9 +29,8 @@ class QuicMultipathReceivedPacketManagerPeer {
static void SetPathReceivedPacketManager(
QuicMultipathReceivedPacketManager* multipath_manager,
QuicPathId path_id,
- QuicReceivedPacketManager* manager) {
- delete multipath_manager->path_managers_[path_id];
- multipath_manager->path_managers_[path_id] = manager;
+ std::unique_ptr<QuicReceivedPacketManager> manager) {
+ multipath_manager->path_managers_[path_id] = std::move(manager);
}
};
@@ -47,9 +47,9 @@ class QuicMultipathReceivedPacketManagerTest : public testing::Test {
manager_0_(new MockReceivedPacketManager(&stats_)),
manager_1_(new MockReceivedPacketManager(&stats_)) {
QuicMultipathReceivedPacketManagerPeer::SetPathReceivedPacketManager(
- &multipath_manager_, kDefaultPathId, manager_0_);
+ &multipath_manager_, kDefaultPathId, base::WrapUnique(manager_0_));
QuicMultipathReceivedPacketManagerPeer::SetPathReceivedPacketManager(
- &multipath_manager_, kPathId1, manager_1_);
+ &multipath_manager_, kPathId1, base::WrapUnique(manager_1_));
}
QuicConnectionStats stats_;
« no previous file with comments | « net/quic/core/quic_multipath_received_packet_manager.cc ('k') | net/quic/test_tools/crypto_test_utils.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698