Use data reduction proxy when managed proxy config returns direct

chrome/browser/net/chrome_network_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/net/chrome_network_delegate.h"
 
 #include <stdlib.h>
 
 #include <vector>
 
@@ skipping 29 matching lines @@
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/resource_request_info.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/net_errors.h"
 #include "net/base/net_log.h"
 #include "net/cookies/canonical_cookie.h"
 #include "net/cookies/cookie_options.h"
 #include "net/http/http_request_headers.h"
 #include "net/http/http_response_headers.h"
+#include "net/proxy/proxy_config.h"
 #include "net/proxy/proxy_info.h"
+#include "net/proxy/proxy_retry_info.h"
 #include "net/proxy/proxy_server.h"
 #include "net/socket_stream/socket_stream.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_context.h"
 
 #if defined(OS_ANDROID)
 #include "chrome/browser/io_thread.h"
 #include "components/precache/content/precache_manager.h"
 #include "components/precache/content/precache_manager_factory.h"
 #endif
@@ skipping 359 matching lines @@
   if (force_safe_search && rv == net::OK && new_url->is_empty())
     safe_search_util::ForceGoogleSafeSearch(request, new_url);
 
   if (connect_interceptor_)
     connect_interceptor_->WitnessURLRequest(request);
 
   return rv;
 }
 
 void ChromeNetworkDelegate::OnResolveProxy(
-    const GURL& url, int load_flags, net::ProxyInfo* result) {
-  if (!on_resolve_proxy_handler_.is_null()) {
+    const GURL& url,
+    int load_flags,
+    const net::ProxyService* proxy_service,
+    net::ProxyInfo* result) {
+  if (!on_resolve_proxy_handler_.is_null() &&
+      !proxy_config_getter_.is_null()) {
+    net::ProxyConfig data_reduction_proxy_config;
+    proxy_config_getter_.Run(&data_reduction_proxy_config);
     on_resolve_proxy_handler_.Run(url, load_flags,
+                                  data_reduction_proxy_config,
+                                  proxy_service->proxy_retry_info(),

[mef, 2014/08/11 19:44:40]

If proxy_service is never NULL, should it be a const reference instead of
pointer?

[bengr, 2014/08/11 23:03:19]

Done.

                                   data_reduction_proxy_params_, result);
   }
 }
 
 void ChromeNetworkDelegate::OnProxyFallback(const net::ProxyServer& bad_proxy,
                                             int net_error,
                                             bool did_fallback) {
   if (data_reduction_proxy_usage_stats_) {
     data_reduction_proxy_usage_stats_->RecordBypassEventHistograms(
         bad_proxy, net_error, did_fallback);
@@ skipping 367 matching lines @@
     data_reduction_proxy::DataReductionProxyRequestType request_type) {
   DCHECK_GE(received_content_length, 0);
   DCHECK_GE(original_content_length, 0);
   StoreAccumulatedContentLength(received_content_length,
                                 original_content_length,
                                 request_type,
                                 reinterpret_cast<Profile*>(profile_));
   received_content_length_ += received_content_length;
   original_content_length_ += original_content_length;
 }