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

Unified Diff: chrome/browser/resources/options2/chromeos/proxy_options.js

Issue 9513016: uber managed pref banners (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/options2/chromeos/proxy_options.js
diff --git a/chrome/browser/resources/options2/chromeos/proxy_options.js b/chrome/browser/resources/options2/chromeos/proxy_options.js
index c75d035b0b8f69bf17aebbbb900ac7ed347f06c6..e18c47a886cb4304940650cdd292626f80be0319 100644
--- a/chrome/browser/resources/options2/chromeos/proxy_options.js
+++ b/chrome/browser/resources/options2/chromeos/proxy_options.js
@@ -100,7 +100,7 @@ cr.define('options', function() {
* @private
*/
updateBannerVisibility_: function() {
- var bannerDiv = $('info-banner');
+ var bannerDiv = $('managed-prefs-banner');
// Remove class and listener for click event in case they were added
// before and updateBannerVisibility_ is called repeatedly.
bannerDiv.classList.remove('clickable');
@@ -109,8 +109,6 @@ cr.define('options', function() {
// Show banner and determine its message if necessary.
var controlledBy = $('directProxy').controlledBy;
if (controlledBy == '') {
- bannerDiv.hidden = true;
- } else {
bannerDiv.hidden = false;
// controlledBy must match strings loaded in proxy_handler.cc and
// set in proxy_cros_settings_provider.cc.

Powered by Google App Engine
This is Rietveld 408576698