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

Unified Diff: net/base/test_proxy_delegate.h

Issue 1684123004: Bypass the DataReductionProxy for all POST requests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reorder parameters and change "" to std::string() Created 4 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: net/base/test_proxy_delegate.h
diff --git a/net/base/test_proxy_delegate.h b/net/base/test_proxy_delegate.h
index a6698c6b8cf4999d56bae8a273c71d88cda33f7d..93db74e41283f23df0836894f09fa05f4722a14f 100644
--- a/net/base/test_proxy_delegate.h
+++ b/net/base/test_proxy_delegate.h
@@ -44,23 +44,25 @@ class TestProxyDelegate : public ProxyDelegate {
void VerifyOnTunnelRequestCompleted(const std::string& endpoint,
const std::string& proxy_server) const;
void VerifyOnTunnelHeadersReceived(const std::string& origin,
const std::string& proxy_server,
const std::string& status_line) const;
// ProxyDelegate implementation:
void OnResolveProxy(const GURL& url,
+ const std::string& method,
int load_flags,
const ProxyService& proxy_service,
ProxyInfo* result) override;
+
bengr 2016/02/21 21:36:39 Remove the blank line.
RyanSturm 2016/02/26 22:16:18 Done.
void OnTunnelConnectCompleted(const HostPortPair& endpoint,
const HostPortPair& proxy_server,
int net_error) override;
void OnFallback(const ProxyServer& bad_proxy, int net_error) override;
void OnBeforeSendHeaders(URLRequest* request,
const ProxyInfo& proxy_info,
HttpRequestHeaders* headers) override;
void OnBeforeTunnelRequest(const HostPortPair& proxy_server,
HttpRequestHeaders* extra_headers) override;
void OnTunnelHeadersReceived(
@@ -76,11 +78,11 @@ class TestProxyDelegate : public ProxyDelegate {
net::ProxyServer trusted_spdy_proxy_;
HostPortPair on_tunnel_request_completed_endpoint_;
HostPortPair on_tunnel_request_completed_proxy_server_;
HostPortPair on_tunnel_headers_received_origin_;
HostPortPair on_tunnel_headers_received_proxy_server_;
std::string on_tunnel_headers_received_status_line_;
};
} // namespace net
-#endif // NET_BASE_TEST_PROXY_DELEGATE_H_
+#endif // NET_BASE_TEST_PROXY_DELEGATE_H_
bengr 2016/02/21 21:36:39 Did you add whitespace here?
RyanSturm 2016/02/26 22:16:18 Now that I'm not using Eclipse the auto-formatting

Powered by Google App Engine
This is Rietveld 408576698