| commit | 83a189f34e8b84d6a3c3cee2a42ceec8dc60dbdd | |
| author | bnc <bnc@chromium.org> | |
| Fri, 7 Nov 2014 20:29:43 +0000 (7 12:29 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Fri, 7 Nov 2014 20:30:58 +0000 (7 20:30 +0000) | ||
| tree | 72ccbb7ed8bda533887d61dae0cd9640f0458d47 | treesnapshot (tar.gz zip) |
| parent | ce072870c3bae8eccdfaa14e3bd9570ca2de20bf | commitdiff |
Report HTTP/2 HEADERS frames as HEADERS, not SYN_STREAM.
Previously, Framer passed HTTP/2 HEADERS frames with priority as SYN_STREAM to the
visitor. This is not correct because (1) we could get headers opening a stream
with no priority set, (2) HTTP/2 does not have SYN_STREAM frames anyway.
This CL makes the Framer report HTTP/2 HEADERS as HEADERS, and pass priority along. SPDY/3 functionality is unchanged.
This CL includes server change 79116025 by mlavan (changes to spdy_framer* expect for Patch Set 3), and server change 79432157 by bnc (Patch Set 3).
BUG=345769
Review URL: https://codereview.chromium.org/698323002
Cr-Commit-Position: refs/heads/master@{#303285}
Previously, Framer passed HTTP/2 HEADERS frames with priority as SYN_STREAM to the
visitor. This is not correct because (1) we could get headers opening a stream
with no priority set, (2) HTTP/2 does not have SYN_STREAM frames anyway.
This CL makes the Framer report HTTP/2 HEADERS as HEADERS, and pass priority along. SPDY/3 functionality is unchanged.
This CL includes server change 79116025 by mlavan (changes to spdy_framer* expect for Patch Set 3), and server change 79432157 by bnc (Patch Set 3).
BUG=345769
Review URL: https://codereview.chromium.org/698323002
Cr-Commit-Position: refs/heads/master@{#303285}
15 files changed: