diff options
author | ryansturm <ryansturm@chromium.org> | 2016-03-03 15:41:07 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-03 23:43:53 +0000 |
commit | 4bab0683f19f749d1974944281c07abbc0d814cf (patch) | |
tree | 0029717741a2fabbdf00022f0ee0588ab5cfa64a /chromeos | |
parent | 981e50f65b63f960783c8318c2fbc5ad02dc71b7 (diff) | |
download | chromium_src-4bab0683f19f749d1974944281c07abbc0d814cf.zip chromium_src-4bab0683f19f749d1974944281c07abbc0d814cf.tar.gz chromium_src-4bab0683f19f749d1974944281c07abbc0d814cf.tar.bz2 |
Bypass the DataReductionProxy for all POST requests
POST requests are non-dempotent, so clients using the Data Reduction Proxy (DRP) can often end up at an error page due to the fact that if the DRP has any problems connecting to the origin, DRP and the client will not be safely able to determine if the post successfully made it to the server. The impact of this change on data compression should be very limited, and it will improve the client's experience.
BUG=581750
Review URL: https://codereview.chromium.org/1684123004
Cr-Commit-Position: refs/heads/master@{#379137}
Diffstat (limited to 'chromeos')
-rw-r--r-- | chromeos/dbus/services/proxy_resolution_service_provider.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/chromeos/dbus/services/proxy_resolution_service_provider.cc b/chromeos/dbus/services/proxy_resolution_service_provider.cc index ec1a754..44d4e83 100644 --- a/chromeos/dbus/services/proxy_resolution_service_provider.cc +++ b/chromeos/dbus/services/proxy_resolution_service_provider.cc @@ -4,6 +4,7 @@ #include "chromeos/dbus/services/proxy_resolution_service_provider.h" +#include <string> #include <utility> #include "base/bind.h" @@ -133,8 +134,9 @@ class ProxyResolverImpl : public ProxyResolverInterface { base::Unretained(request), origin_thread); const int result = proxy_service->ResolveProxy( - GURL(request->source_url_), net::LOAD_NORMAL, &request->proxy_info_, - completion_callback, NULL, NULL, net::BoundNetLog()); + GURL(request->source_url_), std::string(), net::LOAD_NORMAL, + &request->proxy_info_, completion_callback, NULL, NULL, + net::BoundNetLog()); if (result != net::ERR_IO_PENDING) { VLOG(1) << "Network proxy resolution completed synchronously."; completion_callback.Run(result); |