SPDY code cleanup. No intended functional change.
commitb1178b4afde491cb250c0d68e2807e5772dbb4df
authorbnc <bnc@chromium.org>
Thu, 22 Jan 2015 01:20:57 +0000 (21 17:20 -0800)
committerCommit bot <commit-bot@chromium.org>
Thu, 22 Jan 2015 01:21:37 +0000 (22 01:21 +0000)
tree80d4ab0c6bf53cea52a7bace4a35c002c201023d
parentdde405ee64e9cf1dab9bf38995acef27014de2f9
SPDY code cleanup. No intended functional change.

Remove the StreamId field from SpdyHeadersHandlerInterface methods. The caller
of those methods should validate StreamId before calling
SpdyHeadersHandlerInterface methods. This change makes
SpdyHeadersHandlerInterface compatible with HpackDecoder. HPACK has no notion of
stream ID, so the decoder cannot call e.g. OnHeader() with a stream ID.

Also replaces some CHECKs with LOG(DFATAL)s.

This lands server changes 83639565 by birenroy and 84396743 by bnc.

BUG=345769

Review URL: https://codereview.chromium.org/851163005

Cr-Commit-Position: refs/heads/master@{#312514}
net/spdy/spdy_headers_block_parser.cc
net/spdy/spdy_headers_block_parser.h
net/spdy/spdy_headers_block_parser_test.cc