commit d40f903268521fb0055feae9163b1716c351f14e
author rch <rch@google.com> Mon Sep 23 07:27:04 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Sep 24 08:32:38 2019 -0700
tree 3477d6ada8c4fadec5b45134e0c6e2f1023ae6cb
parent 1bfe0d755973b49c69963834ce99fb881eca833c

Clean up UberQuicStreamIdManagerTest by using a Delegate instead of a full Session. Also change the UberQuicStreamIdManager constructor to take a Delegate, Perspective and Version instead of a session.

gfe-relnote: change the UberQuicStreamIdManager constructor to take a Delegate, Perspective and Version instead of a session. Refactor only. Not flag protected.
PiperOrigin-RevId: 270671933
Change-Id: Ie9f8d6051c115f6076179cf8d1fceac1cf6c989b

quic/core/uber_quic_stream_id_manager.cc

diff --git a/quic/core/uber_quic_stream_id_manager.cc b/quic/core/uber_quic_stream_id_manager.cc
index 98c6e8c..f258384 100644
--- a/quic/core/uber_quic_stream_id_manager.cc
+++ b/quic/core/uber_quic_stream_id_manager.cc
@@ -10,24 +10,26 @@
 namespace quic {
 
 UberQuicStreamIdManager::UberQuicStreamIdManager(
-    QuicSession* session,
+    Perspective perspective,
+    ParsedQuicVersion version,
+    QuicStreamIdManager::DelegateInterface* delegate,
     QuicStreamCount num_expected_unidirectiona_static_streams,
     QuicStreamCount max_open_outgoing_bidirectional_streams,
     QuicStreamCount max_open_outgoing_unidirectional_streams,
     QuicStreamCount max_open_incoming_bidirectional_streams,
     QuicStreamCount max_open_incoming_unidirectional_streams)
-    : bidirectional_stream_id_manager_(session,
+    : bidirectional_stream_id_manager_(delegate,
                                        /*unidirectional=*/false,
-                                       session->perspective(),
-                                       session->transport_version(),
+                                       perspective,
+                                       version.transport_version,
                                        0,
                                        max_open_outgoing_bidirectional_streams,
                                        max_open_incoming_bidirectional_streams),
       unidirectional_stream_id_manager_(
-          session,
+          delegate,
           /*unidirectional=*/true,
-          session->perspective(),
-          session->transport_version(),
+          perspective,
+          version.transport_version,
           num_expected_unidirectiona_static_streams,
           max_open_outgoing_unidirectional_streams,
           max_open_incoming_unidirectional_streams) {}