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

Unified Diff: chrome/browser/resources/options/chromeos/internet_options.js

Issue 8102019: redesign and reimplement proxy config service and tracker, revise proxy ui on cros (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options/chromeos/internet_options.js
===================================================================
--- chrome/browser/resources/options/chromeos/internet_options.js (revision 108608)
+++ chrome/browser/resources/options/chromeos/internet_options.js (working copy)
@@ -455,20 +455,9 @@
$('ipTypeDHCPDiv').hidden = !data.showStaticIPConfig;
$('ipTypeStaticDiv').hidden = !data.showStaticIPConfig;
- // Hide change-proxy-button if proxy is not configurable.
- $('change-proxy-button').hidden = !data.proxyConfigurable;
- // If necessary, set text for change-proxy-text and show it.
- var changeProxyText = $('change-proxy-text');
- if (data.changeProxyText != '') {
- changeProxyText.textContent =
- localStrings.getString(data.changeProxyText);
- changeProxyText.hidden = false;
- } else {
- changeProxyText.hidden = true;
- }
- // Hide change-proxy-section if button and text are hidden.
- $('change-proxy-section').hidden = !data.proxyConfigurable &&
- changeProxyText.hidden;
+ // Hide change-proxy-button and change-proxy-section if not showing proxy.
+ $('change-proxy-button').hidden = !data.showProxy;
+ $('change-proxy-section').hidden = !data.showProxy;
var ipConfigList = $('ipConfigList');
ipConfigList.disabled = $('ipTypeDHCP').checked || !data.showStaticIPConfig;

Powered by Google App Engine
This is Rietveld 408576698