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

Side by Side Diff: net/http/http_stream_parser.h

Issue 2298823002: Resetting the HttpRequestInfo pointers in HttpNetworkTransaction and streams (Closed)
Patch Set: Rebased till refs/heads/master@{#425377} Created 4 years, 2 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 unified diff | Download patch
« no previous file with comments | « net/http/http_stream.h ('k') | net/http/http_stream_parser.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_HTTP_HTTP_STREAM_PARSER_H_ 5 #ifndef NET_HTTP_HTTP_STREAM_PARSER_H_
6 #define NET_HTTP_HTTP_STREAM_PARSER_H_ 6 #define NET_HTTP_HTTP_STREAM_PARSER_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
197 197
198 // Uploads statistics about status line compliance with RFC 7230. 198 // Uploads statistics about status line compliance with RFC 7230.
199 void ValidateStatusLine(const std::string& status_line); 199 void ValidateStatusLine(const std::string& status_line);
200 200
201 // Check if buffers used to send the request are empty. 201 // Check if buffers used to send the request are empty.
202 bool SendRequestBuffersEmpty(); 202 bool SendRequestBuffersEmpty();
203 203
204 // Next state of the request, when the current one completes. 204 // Next state of the request, when the current one completes.
205 State io_state_; 205 State io_state_;
206 206
207 // The request to send. 207 // Null when read state machine is invoked.
208 const HttpRequestInfo* request_; 208 const HttpRequestInfo* request_;
209 209
210 // The request header data. May include a merged request body. 210 // The request header data. May include a merged request body.
211 scoped_refptr<DrainableIOBuffer> request_headers_; 211 scoped_refptr<DrainableIOBuffer> request_headers_;
212 212
213 // Size of just the request headers. May be less than the length of 213 // Size of just the request headers. May be less than the length of
214 // |request_headers_| if the body was merged with the headers. 214 // |request_headers_| if the body was merged with the headers.
215 int request_headers_length_; 215 int request_headers_length_;
216 216
217 // True if HTTP/0.9 should be permitted on non-default ports. 217 // True if HTTP/0.9 should be permitted on non-default ports.
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
289 int upload_error_; 289 int upload_error_;
290 290
291 base::WeakPtrFactory<HttpStreamParser> weak_ptr_factory_; 291 base::WeakPtrFactory<HttpStreamParser> weak_ptr_factory_;
292 292
293 DISALLOW_COPY_AND_ASSIGN(HttpStreamParser); 293 DISALLOW_COPY_AND_ASSIGN(HttpStreamParser);
294 }; 294 };
295 295
296 } // namespace net 296 } // namespace net
297 297
298 #endif // NET_HTTP_HTTP_STREAM_PARSER_H_ 298 #endif // NET_HTTP_HTTP_STREAM_PARSER_H_
OLDNEW
« no previous file with comments | « net/http/http_stream.h ('k') | net/http/http_stream_parser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698