Bypass the DataReductionProxy for all POST requests

content/browser/resolve_proxy_msg_helper.cc

Index: content/browser/resolve_proxy_msg_helper.cc
diff --git a/content/browser/resolve_proxy_msg_helper.cc b/content/browser/resolve_proxy_msg_helper.cc
index e189f1c83d3bd323dad707c38bad19d42c538f1c..7b1e04ed0bc975bf3e16d713f63c3a91f1d92f9d 100644
--- a/content/browser/resolve_proxy_msg_helper.cc
+++ b/content/browser/resolve_proxy_msg_helper.cc
@@ -85,21 +85,21 @@ void ResolveProxyMsgHelper::StartPendingRequest() {
   // Verify the request wasn't started yet.
   DCHECK(NULL == req.pac_req);
 
   if (context_getter_.get()) {
     proxy_service_ = context_getter_->GetURLRequestContext()->proxy_service();
     context_getter_ = NULL;
   }
 
   // Start the request.
   int result = proxy_service_->ResolveProxy(
-      req.url, net::LOAD_NORMAL, &proxy_info_,
+      req.url, std::string(), net::LOAD_NORMAL, &proxy_info_,

[bengr, 2016/02/21 21:36:39]

Why not always pass in the method?

[RyanSturm, 2016/02/26 22:16:18]

This would involve massive refactoring to get the method from either Pepper or
WebPlugin, and it would provide little benefit right now, as this call does not
pass in a ProxyDelegate, which is the only place that the method is consumed.

       base::Bind(&ResolveProxyMsgHelper::OnResolveProxyCompleted,
                  base::Unretained(this)),
       &req.pac_req, NULL, net::BoundNetLog());
 
   // Completed synchronously.
   if (result != net::ERR_IO_PENDING)
     OnResolveProxyCompleted(result);
 }
 
 }  // namespace content