diff options
author | rch@chromium.org <rch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-11 00:24:40 +0000 |
---|---|---|
committer | rch@chromium.org <rch@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-11 00:24:40 +0000 |
commit | 3deb9a51f053fd3ddb81222c0a95108ef15630bf (patch) | |
tree | e1d03e98970118a695a8a62e7bf72ce89328b780 /net/http/http_proxy_client_socket.h | |
parent | cbc67383d70e1cd05f49f2281c22700e88615531 (diff) | |
download | chromium_src-3deb9a51f053fd3ddb81222c0a95108ef15630bf.zip chromium_src-3deb9a51f053fd3ddb81222c0a95108ef15630bf.tar.gz chromium_src-3deb9a51f053fd3ddb81222c0a95108ef15630bf.tar.bz2 |
Change the way request headers are logged to the NetLog to ensure
that the complete request is logged, not just the extra_headers.
Before I broke things, HttpNetworkTransaction was attempting to
do the logging, but after some refactors it no longer has access
to the final request. SpdySession already logs correctly in the
SPDY case, and I modified HttpBasicStream to log in the HTTP case.
Added unit tests is both HttpNetworkTransactionTest and
SpdyNetworkTransactionTest.
BUG=48962
TEST=net_unittests --gtest_filter=HttpNetworkTransactionTest.SimpleGET\*
Review URL: http://codereview.chromium.org/4644003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65736 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_proxy_client_socket.h')
-rw-r--r-- | net/http/http_proxy_client_socket.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/http/http_proxy_client_socket.h b/net/http/http_proxy_client_socket.h index b42c78b..e11e731 100644 --- a/net/http/http_proxy_client_socket.h +++ b/net/http/http_proxy_client_socket.h @@ -156,7 +156,8 @@ class HttpProxyClientSocket : public ClientSocket { // If true, then the connection to the proxy is a SPDY connection. const bool using_spdy_; - std::string request_headers_; + std::string request_line_; + HttpRequestHeaders request_headers_; const BoundNetLog net_log_; |