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

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

Issue 2417183003: Remove stl_util's deletion functions from remaining quic code. (Closed)
Patch Set: rebase 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
Index: net/quic/core/quic_sent_packet_manager_test.cc
diff --git a/net/quic/core/quic_sent_packet_manager_test.cc b/net/quic/core/quic_sent_packet_manager_test.cc
index 62e5993ac136d8b0773d4536a6e6c9d6e3002d09..3e03eaf26d26d6b30dd64e3438b7f3fe815b431a 100644
--- a/net/quic/core/quic_sent_packet_manager_test.cc
+++ b/net/quic/core/quic_sent_packet_manager_test.cc
@@ -5,8 +5,8 @@
#include "net/quic/core/quic_sent_packet_manager.h"
#include <memory>
+
#include "base/memory/ptr_util.h"
-#include "base/stl_util.h"
#include "net/quic/core/quic_flags.h"
#include "net/quic/test_tools/quic_config_peer.h"
#include "net/quic/test_tools/quic_sent_packet_manager_peer.h"
@@ -106,7 +106,7 @@ class QuicSentPacketManagerTest : public ::testing::TestWithParam<TestParams> {
.Times(AnyNumber());
}
- ~QuicSentPacketManagerTest() override { base::STLDeleteElements(&packets_); }
+ ~QuicSentPacketManagerTest() override {}
QuicByteCount BytesInFlight() {
return QuicSentPacketManagerPeer::GetBytesInFlight(&manager_);
@@ -305,7 +305,6 @@ class QuicSentPacketManagerTest : public ::testing::TestWithParam<TestParams> {
QuicFlagSaver flags_; // Save/restore all QUIC flag values.
QuicSentPacketManager manager_;
- vector<QuicEncryptedPacket*> packets_;
MockClock clock_;
QuicConnectionStats stats_;
MockSendAlgorithm* send_algorithm_;
« no previous file with comments | « net/quic/core/crypto/crypto_server_config_protobuf.cc ('k') | net/quic/core/quic_unacked_packet_map_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698