Make the transactions own the HttpResponseInfo.

Necessary since we need the SSLInfo to handle certificate errors, but it lives within the HttpResponseInfo.  SSL is before we choose http or spdy, so we don't have an http stream or a spdy stream yet, so they cannot own the HttpResponseInfo.

Review URL: http://codereview.chromium.org/500039

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34773 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 3 deletions

diff --git a/net/http/http_stream_parser.h b/net/http/http_stream_parser.h
index f2108a9..4ccc6ee 100644
--- a/net/http/http_stream_parser.h
+++ b/net/http/http_stream_parser.h
@@ -33,7 +33,8 @@ class HttpStreamParser {
   // These functions implement the interface described in HttpStream with
   // some additional functionality
   int SendRequest(const HttpRequestInfo* request, const std::string& headers,
-                  UploadDataStream* request_body, CompletionCallback* callback);
+                  UploadDataStream* request_body, HttpResponseInfo* response,
+                  CompletionCallback* callback);
 
   int ReadResponseHeaders(CompletionCallback* callback);
 
@@ -128,8 +129,8 @@ class HttpStreamParser {
   // -1 if not found yet.
   int response_header_start_offset_;
 
-  // The parsed response headers.
-  HttpResponseInfo response_;
+  // The parsed response headers.  Owned by the caller.
+  HttpResponseInfo* response_;
 
   // Indicates the content length.  If this value is less than zero
   // (and chunked_decoder_ is null), then we must read until the server