tree da83e4e100607a750706e7b5f6dc4a6a15344a34
parent 2061ddf0e52339e2d4ed6d9c52ce6f3ad1920ceb
author QUICHE team <quiche-dev@google.com> 1640037900 -0800
committer Copybara-Service <copybara-worker@google.com> 1640037963 -0800

Avoid sending frames after sending an immediate GOAWAY in OgHttp2Session.

This CL makes the following related changes to OgHttp2Session:
  - Add a queued_immediate_goaway_ field, set if the enqueued GOAWAY is the
    result of a connection-level error (i.e., if latched_error_ is true).
  - Make EnqueueFrame() a no-op if queued_immediate_goaway_ is true.
  - Avoid writing for streams if queued_immediate_goaway_ is true.

These changes allow OgHttp2Session to avoid sending either control frames or
DATA frames once a GOAWAY is sent.

This CL also fixes a potential memory leak in NgHttp2Adapter with submitted
METADATA that is now also not sent after an immediate GOAWAY. In particular,
MetadataSource now has a new interface method OnFailure() that
SelfDeletingMetadataSource() can override to self-delete.
MetadataSource::OnFailure() is now called in
nghttp2_on_frame_not_send_callback() (newly set to call new OnFrameNotSent()).

This CL marks the triumphant passing of codec_impl_test
LargeServerBodyFlushTimeoutAfterGoaway
(http://sponge2/cc0358b2-2881-4502-aa95-6cf2cd9b6de5).

PiperOrigin-RevId: 417475633