From 90e70b8caf26b73241c6f002c53d8cc96b6496b5 Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Thu, 22 Feb 2018 13:48:06 +0100 Subject: [PATCH] http2: simplify timeout tracking MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There’s no need to reset the chunk counter for every write. PR-URL: https://github.com/nodejs/node/pull/19206 Reviewed-By: James M Snell Reviewed-By: Daniel Bevenius Reviewed-By: Anatoli Papirovski Reviewed-By: Colin Ihrig --- src/node_http2.cc | 7 ------- src/node_http2.h | 2 -- 2 files changed, 9 deletions(-) diff --git a/src/node_http2.cc b/src/node_http2.cc index 4ec6e6e54f797e..d683b075bc2ed3 100644 --- a/src/node_http2.cc +++ b/src/node_http2.cc @@ -1630,10 +1630,6 @@ inline Http2Stream* Http2Session::SubmitRequest( return stream; } -inline void Http2Session::SetChunksSinceLastWrite(size_t n) { - chunks_sent_since_last_write_ = n; -} - // Callback used to receive inbound data from the i/o stream void Http2Session::OnStreamRead(ssize_t nread, const uv_buf_t& buf) { Http2Scope h2scope(this); @@ -2016,7 +2012,6 @@ inline int Http2Stream::DoWrite(WriteWrap* req_wrap, CHECK(!this->IsDestroyed()); CHECK_EQ(send_handle, nullptr); Http2Scope h2scope(this); - session_->SetChunksSinceLastWrite(); if (!IsWritable()) { req_wrap->Done(UV_EOF); return 0; @@ -2519,8 +2514,6 @@ void Http2Stream::RespondFD(const FunctionCallbackInfo& args) { int64_t length = args[3]->IntegerValue(context).ToChecked(); int options = args[4]->IntegerValue(context).ToChecked(); - stream->session()->SetChunksSinceLastWrite(); - Headers list(isolate, context, headers); args.GetReturnValue().Set(stream->SubmitFile(fd, *list, list.length(), offset, length, options)); diff --git a/src/node_http2.h b/src/node_http2.h index 8f6662a0160bec..08109dcf046ba4 100644 --- a/src/node_http2.h +++ b/src/node_http2.h @@ -884,8 +884,6 @@ class Http2Session : public AsyncWrap, public StreamListener { // Write data to the session inline ssize_t Write(const uv_buf_t* bufs, size_t nbufs); - inline void SetChunksSinceLastWrite(size_t n = 0); - size_t self_size() const override { return sizeof(*this); } inline void GetTrailers(Http2Stream* stream, uint32_t* flags);