mirror/naiveproxy
1
0
Fork 0
mirror of https://github.com/klzgrad/naiveproxy.git synced 2024-11-24 14:26:09 +03:00
Code Issues Projects Releases Wiki Activity

h2: Notify delegate about read EOF

Browse Source 
So the delegate can close the socket instead of keeping sending data.

Read EOF or h2 half-closed (remote) state was introduced in
https://codereview.chromium.org/129543002. But StreamSocket doesnt
really supports a half closed state, so upon a read EOF the only sane
action is to close the socket immediately even if in theory more send
is possible.
This commit is contained in:
klzgrad 2018-01-24 23:58:24 -05:00
parent 6027aace34
commit 8593a2d037
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
net/spdy/chromium/spdy_stream.cc
View File

@ -509,6 +509,7 @@ void SpdyStream::OnDataReceived(std::unique_ptr<SpdyBuffer> buffer) {
if (!buffer) { if (!buffer) {
if (io_state_ == STATE_OPEN) { if (io_state_ == STATE_OPEN) {
io_state_ = STATE_HALF_CLOSED_REMOTE; io_state_ = STATE_HALF_CLOSED_REMOTE;
delegate_->OnDataReceived(nullptr);
} else if (io_state_ == STATE_HALF_CLOSED_LOCAL) { } else if (io_state_ == STATE_HALF_CLOSED_LOCAL) {
io_state_ = STATE_CLOSED; io_state_ = STATE_CLOSED;
// Deletes |this|. // Deletes |this|.