Resetting the HttpRequestInfo pointers in HttpNetworkTransaction and streams

net/http/http_basic_state.h

Index: net/http/http_basic_state.h
diff --git a/net/http/http_basic_state.h b/net/http/http_basic_state.h
index d9d66f6ffc99ad632c26391e412f560b348d5b8c..aeff6add963d08a9ee5c0d6f04ac0d50c06cc824 100644
--- a/net/http/http_basic_state.h
+++ b/net/http/http_basic_state.h
@@ -16,6 +16,7 @@
 #include "net/base/completion_callback.h"
 #include "net/base/net_export.h"
 #include "net/base/request_priority.h"
+#include "url/gurl.h"
 
 namespace net {
 
@@ -63,7 +64,8 @@ class NET_EXPORT_PRIVATE HttpBasicState {
 
   const bool using_proxy_;
 
-  const HttpRequestInfo* request_info_;
+  GURL url_;
+  std::string request_method_;

[mmenke, 2016/08/31 21:57:59]

Why do we need these?  Generating the request line after the HttpRequestInfo is
cleared is useless anyways - we no longer have the request body to upload.

[shivanisha, 2016/09/08 20:43:54]

We need these only for generating request line. I have saved these instead of
request info pointer as request info is not needed here leading to one less case
to worry about for resetting.

 
   DISALLOW_COPY_AND_ASSIGN(HttpBasicState);
 };