Chromium Code Reviews
DescriptionAdd SpdyFramerTests for existing ALTSVC handling functionality.
Add test for currently existing functionality (where "currently"
means since https://crrev.com/2723203002 landed earlier today):
* ALTSVC frames with empty or non-empty origins on stream zero
or other streams are processed by the framer. Even though
RFC7838 requires that these frames are ignored, this is done
in a different layer.
* On the other hand, ALTSVC frames with invalid
Alt-Svc-Field-Value result in an error.
These tests are parametrized, and are run both using SpdyFramer
(the old frame decoder) and Http2FrameDecoder (the new frame
decoder). This ensures that behavior of the two decoders is
consistent.
This CL lands server change 148902601 by bnc.
BUG=488484, 696653
Review-Url: https://codereview.chromium.org/2725803004
Cr-Commit-Position: refs/heads/master@{#454138}
Committed: https://chromium.googlesource.com/chromium/src/+/9b550534ae5ce2e122e3a96086cd995070deddc4
Patch Set 1 #
Messages
Total messages: 12 (8 generated)
|
|||||||||||||||||||