Clear QuicChromiumPacketReaders before closing sockets in QuicChromiu…

…mClientSession

QuicChromiumPacketReader must not outlive DatagramClientSocket
because readers have unowned raw pointers to sockets.

Bug: 1449478
Change-Id: I9e28d3a0617e27614451e3b2d8e4cf3e2e6d3145
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4572060
Reviewed-by: Nidhi Jaju <nidhijaju@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1150218}

net/quic/quic_chromium_client_session.cc

@@ -2129,6 +2129,10 @@ void QuicChromiumClientSession::OnConnectionClosed(
   }
 
   CHECK_EQ(sockets_.size(), packet_readers_.size());
+  // Packet readers must not outlive sockets. Clear them before deleting sockets
+  // below.
+  packet_readers_.clear();
+
   bool socket_found_in_writer = false;
   for (auto& socket : sockets_) {
     socket->Close();