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

Unified Diff: net/quic/core/quic_session.h

Issue 2916233002: Remove is_deletable_ from QuicStream. Add QuicStream::IsWaitingForAcks (Closed)
Patch Set: Created 3 years, 7 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
« no previous file with comments | « no previous file | net/quic/core/quic_session.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_session.h
diff --git a/net/quic/core/quic_session.h b/net/quic/core/quic_session.h
index 2a0803d434ee3b4b706a28e5a416ad67e3cfcb97..d70e4bdf6349ec74201df719d4d2fb37c55e05b2 100644
--- a/net/quic/core/quic_session.h
+++ b/net/quic/core/quic_session.h
@@ -215,8 +215,10 @@ class QUIC_EXPORT_PRIVATE QuicSession : public QuicConnectionVisitorInterface,
// WINDOW_UPDATE arrives.
void MarkConnectionLevelWriteBlocked(QuicStreamId id);
- // Called by a stream when it becomes deletable.
- void MarkStreamDeletable(QuicStreamId id);
+ // Called when stream |id| is done waiting for acks either because all data
+ // gets acked or is not interested in data being acked (which happens when
+ // a stream is reset because of an error).
+ void OnStreamDoneWaitingForAcks(QuicStreamId id);
// Returns true if the session has data to be sent, either queued in the
// connection, or in a write-blocked stream.
« no previous file with comments | « no previous file | net/quic/core/quic_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698