35. relnote: Call QuicBufferedPacketStore::MaybeSetExpirationAlarm instead of setting alarm directl…

net/quic/core/quic_buffered_packet_store.h

 // Copyright (c) 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_QUIC_QUIC_BUFFERED_PACKET_STORE_H_
 #define NET_QUIC_QUIC_BUFFERED_PACKET_STORE_H_
 
 #include "net/base/ip_address.h"
 #include "net/base/linked_hash_map.h"
 #include "net/base/net_export.h"
@@ skipping 123 matching lines @@
   void MaybeSetExpirationAlarm();
 
   // Return true if add an extra packet will go beyond allowed max connection
   // limit. The limit for non-CHLO packet and CHLO packet is different.
   bool ShouldBufferPacket(bool is_chlo);
 
   // A map to store packet queues with creation time for each connection.
   BufferedPacketMap undecryptable_packets_;
 
   // The max time the packets of a connection can be buffer in the store.
-  QuicTime::Delta connection_life_span_;
+  const QuicTime::Delta connection_life_span_;
 
   VisitorInterface* visitor_;  // Unowned.
 
   const QuicClock* clock_;  // Unowned.
 
   // This alarm fires every |connection_life_span_| to clean up
   // packets staying in the store for too long.
   std::unique_ptr<QuicAlarm> expiration_alarm_;
 
   // Keeps track of connection with CHLO buffered up already and the order they
   // arrive.
   linked_hash_map<QuicConnectionId, bool> connections_with_chlo_;
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_QUIC_BUFFERED_PACKET_STORE_H_