commit 89490e001b5d2d6c735f2f5d6fc758d4fa5a0301
author wub <wub@google.com> Thu Dec 12 12:45:58 2019 -0800
committer Copybara-Service <copybara-worker@google.com> Thu Dec 12 12:46:57 2019 -0800
tree 0cb549ac9e5f56651de3beb1de7f1d710c445ae4
parent 2f07744895e01b340c0a62b763149dddb13995ed

gfe-relnote: (n/a) Change the return type of QuicDispatcher::CreateQuicSession from raw to unique_ptr. No behavior change, not protected.

PiperOrigin-RevId: 285249381
Change-Id: Ia6d129501364dc231f86f4bda2be36ffd848bb93

quic/test_tools/quic_test_server.cc

diff --git a/quic/test_tools/quic_test_server.cc b/quic/test_tools/quic_test_server.cc
index 3e255b4..15bcae0 100644
--- a/quic/test_tools/quic_test_server.cc
+++ b/quic/test_tools/quic_test_server.cc
@@ -93,7 +93,7 @@
         stream_factory_(nullptr),
         crypto_stream_factory_(nullptr) {}
 
-  QuicServerSessionBase* CreateQuicSession(
+  std::unique_ptr<QuicSession> CreateQuicSession(
       QuicConnectionId id,
       const QuicSocketAddress& client,
       quiche::QuicheStringPiece alpn,
@@ -108,9 +108,9 @@
                            /* owns_writer= */ false, Perspective::IS_SERVER,
                            ParsedQuicVersionVector{version});
 
-    QuicServerSessionBase* session = nullptr;
+    std::unique_ptr<QuicServerSessionBase> session;
     if (stream_factory_ != nullptr || crypto_stream_factory_ != nullptr) {
-      session = new CustomStreamSession(
+      session = std::make_unique<CustomStreamSession>(
           config(), GetSupportedVersions(), connection, this, session_helper(),
           crypto_config(), compressed_certs_cache(), stream_factory_,
           crypto_stream_factory_, server_backend());