| Index: chrome/browser/policy/configuration_policy_pref_store.cc
|
| diff --git a/chrome/browser/policy/configuration_policy_pref_store.cc b/chrome/browser/policy/configuration_policy_pref_store.cc
|
| index 23e65106c3adebf2a9f45feb3ca8c79b8789af5d..6aac1205fa43ad08156ff81ba9be7dce36556705 100644
|
| --- a/chrome/browser/policy/configuration_policy_pref_store.cc
|
| +++ b/chrome/browser/policy/configuration_policy_pref_store.cc
|
| @@ -535,16 +535,16 @@ bool ConfigurationPolicyPrefKeeper::CheckProxySettings() {
|
| }
|
| break;
|
| case kPolicyManuallyConfiguredProxyMode:
|
| - if (!server) {
|
| + if (server && pac_url) {
|
| LOG(WARNING) << "A centrally-administered policy dictates that the"
|
| - << " system proxy settings should use fixed proxy servers"
|
| - << " without specifying which ones.";
|
| + << " system proxy settings should use both a fixed"
|
| + << " proxy server and a .pac url.";
|
| return false;
|
| }
|
| - if (pac_url) {
|
| + if (!server && !pac_url) {
|
| LOG(WARNING) << "A centrally-administered policy dictates that the"
|
| - << " system proxy settings should use fixed proxy servers"
|
| - << " but also specifies a PAC script.";
|
| + << " system proxy settings should use either a fixed"
|
| + << " proxy server or a .pac url, but specifies neither.";
|
| return false;
|
| }
|
| break;
|
| @@ -576,11 +576,11 @@ void ConfigurationPolicyPrefKeeper::ApplyProxySettings() {
|
| break;
|
| case kPolicyAutoDetectProxyMode:
|
| mode = ProxyPrefs::MODE_AUTO_DETECT;
|
| - if (HasProxyPolicy(kPolicyProxyPacUrl))
|
| - mode = ProxyPrefs::MODE_PAC_SCRIPT;
|
| break;
|
| case kPolicyManuallyConfiguredProxyMode:
|
| mode = ProxyPrefs::MODE_FIXED_SERVERS;
|
| + if (HasProxyPolicy(kPolicyProxyPacUrl))
|
| + mode = ProxyPrefs::MODE_PAC_SCRIPT;
|
| break;
|
| case kPolicyUseSystemProxyMode:
|
| mode = ProxyPrefs::MODE_SYSTEM;
|
|
|