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

Unified Diff: net/quic/chromium/quic_http_stream.h

Issue 2298823002: Resetting the HttpRequestInfo pointers in HttpNetworkTransaction and streams (Closed)
Patch Set: Initial patch Created 4 years, 4 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/chromium/quic_http_stream.h
diff --git a/net/quic/chromium/quic_http_stream.h b/net/quic/chromium/quic_http_stream.h
index 79bb187b828d5d33d92fe46fef72521e2e1c80af..47b5b541ac0e4f261ba49e26d8b104625bdb96a9 100644
--- a/net/quic/chromium/quic_http_stream.h
+++ b/net/quic/chromium/quic_http_stream.h
@@ -132,6 +132,8 @@ class NET_EXPORT_PRIVATE QuicHttpStream
void ResetStream();
bool CancelPromiseIfHasBody();
+ void ResetRequestInfo();
+
State next_state_;
base::WeakPtr<QuicChromiumClientSession> session_;
@@ -144,7 +146,12 @@ class NET_EXPORT_PRIVATE QuicHttpStream
// outlive this object, according to the HttpStream contract.
// The request to send.
+ // Only valid before the response body is read.
const HttpRequestInfo* request_info_;
+
+ // upload progress saved while resetting the request info to null.
+ UploadProgress upload_progress_;
+
// The request body to send, if any, owned by the caller.
UploadDataStream* request_body_stream_;
// Time the request was issued.

Powered by Google App Engine
This is Rietveld 408576698