DescriptionRevert of Increase maximum size of the HPACK decoder dynamic table to 64 kB. (patchset #1 id:1 of https://codereview.chromium.org/2300683002/ )
Reason for revert:
Suspected patch causing Android Cronet Builder and other bots to fail BidirectionalStreamTest#testSimpleGet. Will re-land if not.
Original issue's description:
> Increase maximum size of the HPACK decoder dynamic table to 64 kB.
>
> * Send out SETTINGS_HEADER_TABLE_SIZE = 64 kB in the initial SETTINGS frame on
> each HTTP/2 connection.
> * Immediately notify HpackDecoder about the change so that it allows the encoder
> to update the dynamic table size up to this limit. It is safe to do so before
> receiving the SETTINGS ACK, because the new limit is larger than the default 4
> kB. In fact, a server following RFC 7540 Section 6.5.3 word-by-word might
> already use the larger limit before sending an ACK.
>
> BUG=642784
>
> Committed: https://crrev.com/30537e17941018ecffe781c009b39c7a8c4c8f30
> Cr-Commit-Position: refs/heads/master@{#415718}
TBR=rch@chromium.org,bnc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=642784
Committed: https://crrev.com/d9ba3be0a5057b52aa35756a3fe3156301890097
Cr-Commit-Position: refs/heads/master@{#415735}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|