Chromium Code Reviews| Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc |
| diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc |
| index 7fdb3212c0808cfec96cee0502781088d15c8296..e2ec1da8aca8b2fa140b3635378ef07ebbf830bf 100644 |
| --- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc |
| +++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_settings.cc |
| @@ -186,10 +186,16 @@ void DataReductionProxySettings::SetProxyConfigurator( |
| configurator_ = configurator; |
| } |
| -bool DataReductionProxySettings::IsDataReductionProxyEnabled() { |
| +bool DataReductionProxySettings::IsDataReductionProxyEnabled() const { |
| return spdy_proxy_auth_enabled_.GetValue() || IsEnabledOnCommandLine(); |
| } |
| +bool DataReductionProxySettings::CanUseDataReductionProxy(const GURL& url) |
| + const { |
| + return url.is_valid() && (url.scheme() != "https") && |
|
bengr
2015/02/20 01:19:25
why not specifically check that the scheme is http
Not at Google. Contact bengr
2015/02/20 17:08:08
Copied from Android implementation: https://cs.cor
Not at Google. Contact bengr
2015/02/20 18:43:38
Used url::kHttpScheme and changed to '=='.
|
| + IsDataReductionProxyEnabled(); |
| +} |
| + |
| bool |
| DataReductionProxySettings::IsDataReductionProxyAlternativeEnabled() const { |
| return data_reduction_proxy_alternative_enabled_.GetValue(); |