Consolidates bool members of quicspdysession to reduce wasted padding.

PiperOrigin-RevId: 321202984
Change-Id: I2c74e82cba38ad2a5c8edba2239e7b932f5dc0e1

quic/core/http/quic_spdy_session.cc

diff --git a/quic/core/http/quic_spdy_session.cc b/quic/core/http/quic_spdy_session.cc
index b9acfef..ed0469f 100644
--- a/quic/core/http/quic_spdy_session.cc
+++ b/quic/core/http/quic_spdy_session.cc
@@ -394,15 +394,15 @@
           QuicUtils::GetInvalidStreamId(connection->transport_version())),
       promised_stream_id_(
           QuicUtils::GetInvalidStreamId(connection->transport_version())),
-      fin_(false),
       frame_len_(0),
+      fin_(false),
       spdy_framer_(SpdyFramer::ENABLE_COMPRESSION),
       spdy_framer_visitor_(new SpdyFramerVisitor(this)),
+      debug_visitor_(nullptr),
+      destruction_indicator_(123456789),
       server_push_enabled_(true),
       ietf_server_push_enabled_(
           GetQuicFlag(FLAGS_quic_enable_http3_server_push)),
-      destruction_indicator_(123456789),
-      debug_visitor_(nullptr),
       http3_goaway_received_(false),
       http3_goaway_sent_(false),
       http3_max_push_id_sent_(false) {

quic/core/http/quic_spdy_session.h

diff --git a/quic/core/http/quic_spdy_session.h b/quic/core/http/quic_spdy_session.h
index 3773b88..d952566 100644
--- a/quic/core/http/quic_spdy_session.h
+++ b/quic/core/http/quic_spdy_session.h
@@ -535,22 +535,13 @@
   // Data about the stream whose headers are being processed.
   QuicStreamId stream_id_;
   QuicStreamId promised_stream_id_;
-  bool fin_;
   size_t frame_len_;
+  bool fin_;
 
   spdy::SpdyFramer spdy_framer_;
   http2::Http2DecoderAdapter h2_deframer_;
   std::unique_ptr<SpdyFramerVisitor> spdy_framer_visitor_;
 
-  // Used in Google QUIC only.  Set every time SETTINGS_ENABLE_PUSH is received.
-  // Defaults to true.
-  bool server_push_enabled_;
-
-  // Used in IETF QUIC only.  Defaults to false.
-  // Server push is enabled for a server by calling EnableServerPush().
-  // Server push is enabled for a client by calling SetMaxPushId().
-  bool ietf_server_push_enabled_;
-
   // Used in IETF QUIC only.
   // For a server:
   //   the push ID in the most recently received MAX_PUSH_ID frame,
@@ -564,13 +555,26 @@
   // Once set, never goes back to unset.
   quiche::QuicheOptional<PushId> max_push_id_;
 
+  // Not owned by the session.
+  Http3DebugVisitor* debug_visitor_;
+
+  // Priority values received in PRIORITY_UPDATE frames for streams that are not
+  // open yet.
+  QuicHashMap<QuicStreamId, int> buffered_stream_priorities_;
+
   // An integer used for live check. The indicator is assigned a value in
   // constructor. As long as it is not the assigned value, that would indicate
   // an use-after-free.
   int32_t destruction_indicator_;
 
-  // Not owned by the session.
-  Http3DebugVisitor* debug_visitor_;
+  // Used in Google QUIC only.  Set every time SETTINGS_ENABLE_PUSH is received.
+  // Defaults to true.
+  bool server_push_enabled_;
+
+  // Used in IETF QUIC only.  Defaults to false.
+  // Server push is enabled for a server by calling EnableServerPush().
+  // Server push is enabled for a client by calling SetMaxPushId().
+  bool ietf_server_push_enabled_;
 
   // If the endpoint has received HTTP/3 GOAWAY frame.
   bool http3_goaway_received_;
@@ -581,10 +585,6 @@
   // has been sent, in which case |max_push_id_| has the value sent in the most
   // recent MAX_PUSH_ID frame.  Once true, never goes back to false.
   bool http3_max_push_id_sent_;
-
-  // Priority values received in PRIORITY_UPDATE frames for streams that are not
-  // open yet.
-  QuicHashMap<QuicStreamId, int> buffered_stream_priorities_;
 };
 
 }  // namespace quic