Chromium Code Reviews| Index: chrome/browser/resources/options/chromeos/internet_detail.js |
| diff --git a/chrome/browser/resources/options/chromeos/internet_detail.js b/chrome/browser/resources/options/chromeos/internet_detail.js |
| index 28c692cf142352a912eea3ac0bfcffa55b9855cb..4f1f0b920a3ba80b3bd03ea047e49b700360a423 100644 |
| --- a/chrome/browser/resources/options/chromeos/internet_detail.js |
| +++ b/chrome/browser/resources/options/chromeos/internet_detail.js |
| @@ -261,11 +261,14 @@ cr.define('options.internet', function() { |
| $('auto-proxy').addEventListener('click', this.disableManualProxy_); |
| $('proxy-all-protocols').addEventListener('click', |
| this.toggleSingleProxy_); |
| + $('proxy-use-pac-url').addEventListener('click', |
|
Dan Beam
2013/04/17 20:55:50
nit: click -> change
|
| + this.handleAutoConfigProxy_); |
| observePrefsUI($('direct-proxy')); |
| observePrefsUI($('manual-proxy')); |
| observePrefsUI($('auto-proxy')); |
| observePrefsUI($('proxy-all-protocols')); |
| + observePrefsUI($('proxy-use-pac-url')); |
| $('ip-automatic-configuration-checkbox').addEventListener('click', |
| this.handleIpAutoConfig_); |
| @@ -430,6 +433,20 @@ cr.define('options.internet', function() { |
| }, |
| /** |
| + * Handler for when the user clicks on the checkbox to enter |
| + * auto configuration URL. |
| + * @private |
| + * @param {Event} e Click Event. |
| + */ |
| + handleAutoConfigProxy_: function(e) { |
| + if ($('proxy-use-pac-url').checked) { |
| + $('proxy-pac-url').disabled = false; |
| + } else { |
| + $('proxy-pac-url').disabled = true; |
| + } |
|
Dan Beam
2013/04/17 20:55:50
$('proxy-pac-url').disabled = !$('proxy-use-pac-ur
|
| + }, |
| + |
| + /** |
| * Handler for selecting a radio button that will disable the manual |
| * controls. |
| * @private |
| @@ -448,8 +465,10 @@ cr.define('options.internet', function() { |
| $('ftp-proxy-port').disabled = true; |
| $('socks-host').disabled = true; |
| $('socks-port').disabled = true; |
| - $('proxy-config').disabled = $('auto-proxy').disabled || |
| - !$('auto-proxy').checked; |
| + $('proxy-use-pac-url').disabled = $('auto-proxy').disabled || |
| + !$('auto-proxy').checked; |
| + $('proxy-pac-url').disabled = $('proxy-use-pac-url').disabled || |
| + !$('proxy-use-pac-url').checked; |
| }, |
| /** |
| @@ -476,7 +495,8 @@ cr.define('options.internet', function() { |
| $('ftp-proxy-port').disabled = allDisabled; |
| $('socks-host').disabled = allDisabled; |
| $('socks-port').disabled = allDisabled; |
| - $('proxy-config').disabled = true; |
| + $('proxy-use-pac-url').disabled = true; |
| + $('proxy-pac-url').disabled = true; |
| }, |
| }; |