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

Unified Diff: chrome/browser/resources/options2/options_page.js

Issue 9307026: make search box in settings page not get hidden behind enterprise warning (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: .hidden -> [hidden] 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
« no previous file with comments | « chrome/browser/resources/options2/options_page.css ('k') | chrome/browser/resources/options2/pref_ui.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/options2/options_page.js
diff --git a/chrome/browser/resources/options2/options_page.js b/chrome/browser/resources/options2/options_page.js
index 7b16dcc89f86af5ed44e868e2708fbb828eb9e41..8c86f0fafc1981480ab77c9004bbc02d7054dcd7 100644
--- a/chrome/browser/resources/options2/options_page.js
+++ b/chrome/browser/resources/options2/options_page.js
@@ -898,43 +898,6 @@ cr.define('options', function() {
initializePage: function() {},
/**
- * Updates managed banner visibility state. This function iterates over
- * all input fields of a window and if any of these is marked as managed
- * it triggers the managed banner to be visible. The banner can be enforced
- * being on through the managed flag of this class but it can not be forced
- * being off if managed items exist.
- */
- updateManagedBannerVisibility: function() {
- var bannerDiv = $('managed-prefs-banner');
-
- var controlledByPolicy = false;
- var controlledByExtension = false;
- var inputElements = this.pageDiv.querySelectorAll('input[controlled-by]');
- for (var i = 0, len = inputElements.length; i < len; i++) {
- if (inputElements[i].controlledBy == 'policy')
- controlledByPolicy = true;
- else if (inputElements[i].controlledBy == 'extension')
- controlledByExtension = true;
- }
- if (!controlledByPolicy && !controlledByExtension) {
- bannerDiv.hidden = true;
- } else {
- bannerDiv.hidden = false;
- var height = window.getComputedStyle(bannerDiv).height;
- if (controlledByPolicy && !controlledByExtension) {
- $('managed-prefs-text').textContent =
- templateData.policyManagedPrefsBannerText;
- } else if (!controlledByPolicy && controlledByExtension) {
- $('managed-prefs-text').textContent =
- templateData.extensionManagedPrefsBannerText;
- } else if (controlledByPolicy && controlledByExtension) {
- $('managed-prefs-text').textContent =
- templateData.policyAndExtensionManagedPrefsBannerText;
- }
- }
- },
-
- /**
* Gets page visibility state.
*/
get visible() {
@@ -954,11 +917,6 @@ cr.define('options', function() {
OptionsPage.updatePageFreezeStates();
- // The managed prefs banner is global, so after any visibility change
- // update it based on the topmost page, not necessarily this page
- // (e.g., if an ancestor is made visible after a child).
- OptionsPage.updateManagedBannerVisibility();
-
// A subpage was shown or hidden.
if (!this.isOverlay && this.nestingLevel > 0)
OptionsPage.updateDisplayForShowOrHideSubpage_();
« no previous file with comments | « chrome/browser/resources/options2/options_page.css ('k') | chrome/browser/resources/options2/pref_ui.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698