Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(548)

Unified Diff: net/quic/core/quic_flow_controller.cc

Issue 2850573002: Landing Recent QUIC changes until 3:35 PM, Apr 26, 2017 UTC-4 (Closed)
Patch Set: remove reinterpret_cast Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/quic/core/quic_flow_controller.cc
diff --git a/net/quic/core/quic_flow_controller.cc b/net/quic/core/quic_flow_controller.cc
index e95027ebbfb054af7e4cfbd43ab59221cef4a30d..bf36c7a9ab07cb94b2b27b6a2757e1c2668dfbd0 100644
--- a/net/quic/core/quic_flow_controller.cc
+++ b/net/quic/core/quic_flow_controller.cc
@@ -202,10 +202,11 @@ void QuicFlowController::MaybeSendWindowUpdate() {
}
MaybeIncreaseMaxWindowSize();
- SendWindowUpdate(available_window);
+ UpdateReceiveWindowOffsetAndSendWindowUpdate(available_window);
}
-void QuicFlowController::SendWindowUpdate(QuicStreamOffset available_window) {
+void QuicFlowController::UpdateReceiveWindowOffsetAndSendWindowUpdate(
+ QuicStreamOffset available_window) {
// Update our receive window.
receive_window_offset_ += (receive_window_size_ - available_window);
@@ -216,8 +217,7 @@ void QuicFlowController::SendWindowUpdate(QuicStreamOffset available_window) {
<< ", and receive window size: " << receive_window_size_
<< ". New receive window offset is: " << receive_window_offset_;
- // Inform the peer of our new receive window.
- connection_->SendWindowUpdate(id_, receive_window_offset_);
+ SendWindowUpdate();
}
void QuicFlowController::MaybeSendBlocked() {
@@ -265,7 +265,7 @@ void QuicFlowController::EnsureWindowAtLeast(QuicByteCount window_size) {
QuicStreamOffset available_window = receive_window_offset_ - bytes_consumed_;
IncreaseWindowSize();
- SendWindowUpdate(available_window);
+ UpdateReceiveWindowOffsetAndSendWindowUpdate(available_window);
}
bool QuicFlowController::IsBlocked() const {
@@ -292,4 +292,8 @@ void QuicFlowController::UpdateReceiveWindowSize(QuicStreamOffset size) {
receive_window_offset_ = size;
}
+void QuicFlowController::SendWindowUpdate() {
+ connection_->SendWindowUpdate(id_, receive_window_offset_);
+}
+
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698