tree 9e39e41ad2159c96da8bb777758040e1bc424089
parent 1d87a3d3af2f3179e51a67487a866bc00d4f787c
author diannahu <diannahu@google.com> 1662563956 -0700
committer Copybara-Service <copybara-worker@google.com> 1662564087 -0700

Drop buffered metadata for a stream in ForgetStream().

This CL is a follow-up to a comment in cl/471086207
(http://screen/7SxFkVearSiZiaP), which pointed out that buffered metadata for a
stream may still be sent even after ForgetStream() for that stream is called on
the metadata manager.

In practice, this is unlikely to happen in production because pending metadata
is flushed at once for HTTP/2 [1] and will be flushed at once for HTTP/3 [2],
without intervening ForgetStream() calls, but this CL fixes ForgetStream() to
drop buffered metadata for the stream as a solid conceptual fix and in case
future changes allow intervening ForgetStream() calls in HTTP/2 or HTTP/3.

[1] e.g., http://google3/gfe/gfe2/vtl/vtl_client_dispatcher.cc;l=294-302;rcl=464844844
[2] e.g., cl/470321865

PiperOrigin-RevId: 472729185