tree 01b033853f5ebf1a14747baa520a92070f02469a
parent 3120f8fece0462d302a63d402f69d72a8cc550d1
author QUICHE team <quiche-dev@google.com> 1646962750 -0800
committer Copybara-Service <copybara-worker@google.com> 1646962810 -0800

Add Http2DecoderAdapter::SetMaxFrameSize(), currently unused.

This CL performs several minor refactors/non-functional changes:
  - Adds a setter SetMaxFrameSize() to store the new value in
    Http2DecoderAdapter and its Http2FrameDecoder member.
  - Renames recv_frame_size_limit_ to max_frame_size_, to reflect that the
    field corresponds to SETTINGS_MAX_FRAME_SIZE.
  - Removes some unneeded local variables.
  - Removes the frame_decoder_->set_maximum_payload_size() call from
    ProcessInput(). This change is a no-op because
    Http2DecoderAdapter's recv_frame_size_limit_/max_frame_size_ before was not
    altered from its default value [1] (so the repeated calls in ProcessInput()
    were no-ops). Furthermore, the default value already matches the
    frame_decoder_ default value for its maximum_payload_size_ [2].
    In production, ResetInternal() is only called in construction.

The addition of SetMaxFrameSize() is important for a future change in
OgHttp2Session, which will call this new method when sending custom
SETTINGS_MAX_FRAME_SIZE values to the peer.

[1] http://google3/third_party/spdy/core/http2_frame_decoder_adapter.h;l=313;rcl=402940499
[2] http://google3/third_party/http2/decoder/http2_frame_decoder.cc;l=37;rcl=433094660

PiperOrigin-RevId: 433892628