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

Side by Side Diff: chrome/browser/net/chrome_network_delegate.h

Issue 653313003: Check effective proxy config before adding data reduction proxies (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_ 5 #ifndef CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
6 #define CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_ 6 #define CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 // add hooks into the network stack. 71 // add hooks into the network stack.
72 class ChromeNetworkDelegate : public net::NetworkDelegate { 72 class ChromeNetworkDelegate : public net::NetworkDelegate {
73 public: 73 public:
74 // Provides an opportunity to interpose on proxy resolution. Called before 74 // Provides an opportunity to interpose on proxy resolution. Called before
75 // ProxyService.ResolveProxy() returns. |proxy_info| contains information 75 // ProxyService.ResolveProxy() returns. |proxy_info| contains information
76 // about the proxy being used, and may be modified by this callback. 76 // about the proxy being used, and may be modified by this callback.
77 typedef base::Callback<void( 77 typedef base::Callback<void(
78 const GURL& url, 78 const GURL& url,
79 int load_flags, 79 int load_flags,
80 const net::ProxyConfig& data_reduction_proxy_config, 80 const net::ProxyConfig& data_reduction_proxy_config,
81 const net::ProxyRetryInfoMap& proxy_retry_info_map, 81 const net::ProxyConfig& proxy_service_proxy_config,
82 const net::ProxyRetryInfoMap& proxy_retry_info,
mef 2014/10/28 17:01:32 why did you loose |_map|?
bengr 2014/10/29 21:12:36 Done.
82 const data_reduction_proxy::DataReductionProxyParams* params, 83 const data_reduction_proxy::DataReductionProxyParams* params,
83 net::ProxyInfo* result)> OnResolveProxyHandler; 84 net::ProxyInfo* result)> OnResolveProxyHandler;
84 85
85 // Provides an additional proxy configuration that can be consulted after 86 // Provides an additional proxy configuration that can be consulted after
86 // proxy resolution. 87 // proxy resolution.
87 typedef base::Callback<const net::ProxyConfig&()> ProxyConfigGetter; 88 typedef base::Callback<const net::ProxyConfig&()> ProxyConfigGetter;
88 89
89 // |enable_referrers| (and all of the other optional PrefMembers) should be 90 // |enable_referrers| (and all of the other optional PrefMembers) should be
90 // initialized on the UI thread (see below) beforehand. This object's owner is 91 // initialized on the UI thread (see below) beforehand. This object's owner is
91 // responsible for cleaning them up at shutdown. 92 // responsible for cleaning them up at shutdown.
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
335 data_reduction_proxy::DataReductionProxyStatisticsPrefs* 336 data_reduction_proxy::DataReductionProxyStatisticsPrefs*
336 data_reduction_proxy_statistics_prefs_; 337 data_reduction_proxy_statistics_prefs_;
337 338
338 OnResolveProxyHandler on_resolve_proxy_handler_; 339 OnResolveProxyHandler on_resolve_proxy_handler_;
339 ProxyConfigGetter proxy_config_getter_; 340 ProxyConfigGetter proxy_config_getter_;
340 341
341 DISALLOW_COPY_AND_ASSIGN(ChromeNetworkDelegate); 342 DISALLOW_COPY_AND_ASSIGN(ChromeNetworkDelegate);
342 }; 343 };
343 344
344 #endif // CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_ 345 #endif // CHROME_BROWSER_NET_CHROME_NETWORK_DELEGATE_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/net/chrome_network_delegate.cc » ('j') | chrome/browser/net/chrome_network_delegate.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698