tree 11c32d8c29cf2da7415aca0843cc647c6cd1dbe9
parent 7640b5cd7defafe56c69068daabe4e173b53bbbf
author diannahu <diannahu@google.com> 1711635545 -0700
committer Copybara-Service <copybara-worker@google.com> 1711635612 -0700

Latch in SpdyHeadersIR the feature flag that updates SpdyHeadersIR::size().

We observed a b/331280235 coredump with the following Http2WriteQueue member:
  ...
  pending_frames_bytes_ = 18446744073709551614,
  ...

From this value, we found that different values of
--gfe2_reloadable_flag_http2_add_hpack_overhead_bytes at various size() call
sites could lead to inconsistent size estimate values, which would lead to
issues when attempting to write.

The new regression test would have failed using the initial version of this
flag: http://sponge2/ae76de9d-6a60-4031-bc9b-12a95ec6ae47 (failed).

This CL deprecates --gfe2_reloadable_flag_http2_add_hpack_overhead_bytes and
renames the flag with the new behavivor to latch in SpdyHeadersIR.

This CL also changes some Http2WriteQueue member types from size_t to ssize_t.

Protected by FLAGS_gfe2_reloadable_flag_http2_add_hpack_overhead_bytes2.

PiperOrigin-RevId: 619920782