diff options
| author | bengr@chromium.org <bengr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-01 21:20:20 +0000 |
|---|---|---|
| committer | bengr@chromium.org <bengr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-01 21:20:20 +0000 |
| commit | 1252d42fbf09049ce7c0a8812e0c450a62657c61 (patch) | |
| tree | 9c566616e8fd035a7ed72b93f6911c9b5a6bb9d0 /net/http/http_network_transaction_unittest.cc | |
| parent | 2c69b85f1d66c0bc0bdcd6248bf9b7fac9aad7f6 (diff) | |
| download | chromium_src-1252d42fbf09049ce7c0a8812e0c450a62657c61.zip chromium_src-1252d42fbf09049ce7c0a8812e0c450a62657c61.tar.gz chromium_src-1252d42fbf09049ce7c0a8812e0c450a62657c61.tar.bz2 | |
Add header pointer to BeforeProxyHeadersSentCallback
This is needed because the higher layers, e.g., UrlRequestHttpJob
do not always have a pointer to the request headers that the
HttpNetworkTransaction will actually send. Note that the old code
could also pass the wrong headers if redirects resulted in no longer
going through the proxy.
BUG=345907
Review URL: https://codereview.chromium.org/360113004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280885 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_network_transaction_unittest.cc')
| -rw-r--r-- | net/http/http_network_transaction_unittest.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/http/http_network_transaction_unittest.cc b/net/http/http_network_transaction_unittest.cc index 700bc6e..ab64040 100644 --- a/net/http/http_network_transaction_unittest.cc +++ b/net/http/http_network_transaction_unittest.cc @@ -438,7 +438,8 @@ class BeforeProxyHeadersSentHandler { BeforeProxyHeadersSentHandler() : observed_before_proxy_headers_sent_(false) {} - void OnBeforeProxyHeadersSent(const ProxyInfo& proxy_info) { + void OnBeforeProxyHeadersSent(const ProxyInfo& proxy_info, + HttpRequestHeaders* request_headers) { observed_before_proxy_headers_sent_ = true; observed_proxy_server_uri_ = proxy_info.proxy_server().ToURI(); } |