Remove the managed banner from the Chrome settings UI

chrome/browser/resources/options/options_page.js

Index: chrome/browser/resources/options/options_page.js
diff --git a/chrome/browser/resources/options/options_page.js b/chrome/browser/resources/options/options_page.js
index a10ab3c9e557c81b131dd7de85a248452af38e54..bb14e3d4c2e180336aa89b62bccf28fc35d21759 100644
--- a/chrome/browser/resources/options/options_page.js
+++ b/chrome/browser/resources/options/options_page.js
@@ -418,15 +418,6 @@ cr.define('options', function() {
   };
 
   /**
-   * Updates managed banner visibility state based on the topmost page.
-   */
-  OptionsPage.updateManagedBannerVisibility = function() {
-    var topPage = this.getTopmostVisiblePage();
-    if (topPage)
-      topPage.updateManagedBannerVisibility();
-  };
-
-  /**
    * Shows the tab contents for the given navigation tab.
    * @param {!Element} tab The tab that the user clicked.
    */
@@ -715,59 +706,6 @@ cr.define('options', function() {
     initializePage: function() {},
 
     /**
-     * Updates managed banner visibility state. This function iterates over
-     * all input fields of a page 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 = this.pageDiv.querySelector('.managed-prefs-banner');
-      // Create a banner for the overlay if we don't have one.
-      if (!bannerDiv) {
-        bannerDiv = $('managed-prefs-banner').cloneNode(true);
-        bannerDiv.id = null;
-
-        if (this.isOverlay) {
-          var content = this.pageDiv.querySelector('.content-area');
-          content.parentElement.insertBefore(bannerDiv, content);
-        } else {
-          bannerDiv.classList.add('main-page-banner');
-          var header = this.pageDiv.querySelector('header');
-          header.appendChild(bannerDiv);
-        }
-      }
-
-      var controlledByPolicy = false;
-      var controlledByExtension = false;
-      var inputElements = this.pageDiv.querySelectorAll('input[controlled-by]');
-      for (var i = 0; i < inputElements.length; i++) {
-        if (inputElements[i].controlledBy == 'policy')
-          controlledByPolicy = true;
-        else if (inputElements[i].controlledBy == 'extension')
-          controlledByExtension = true;
-      }
-
-      if (!controlledByPolicy && !controlledByExtension) {
-        this.pageDiv.classList.remove('showing-banner');
-      } else {
-        this.pageDiv.classList.add('showing-banner');
-
-        var text = bannerDiv.querySelector('#managed-prefs-text');

[Joao da Silva, 2012/10/25 11:07:52]

An element with id 'managed-prefs-text' exists in
chrome/browser/resources/extensions/extensions.html. Can this code really be
removed?

That element seems related to managed mode, maybe Bernhard knows more about
this.

[bartfab (slow), 2012/10/25 13:13:13]

The managed mode banner actually has a completely independent implementation and
only shared CSS with the options banners. I left the CSS intact, so that banner
is still working (I verified it just in case).

The managed mode banner did have some stale ids/classes that nothing in CSS, JS
or C++ ever referenced. I removed those now.

-        if (controlledByPolicy && !controlledByExtension) {
-          text.textContent =
-              loadTimeData.getString('policyManagedPrefsBannerText');
-        } else if (!controlledByPolicy && controlledByExtension) {
-          text.textContent =
-              loadTimeData.getString('extensionManagedPrefsBannerText');
-        } else if (controlledByPolicy && controlledByExtension) {
-          text.textContent = loadTimeData.getString(
-              'policyAndExtensionManagedPrefsBannerText');
-        }
-      }
-    },
-
-    /**
      * Gets the container div for this page if it is an overlay.
      * @type {HTMLElement}
      */
@@ -892,7 +830,6 @@ cr.define('options', function() {
      */
     onVisibilityChanged_: function() {
       OptionsPage.updateRootPageFreezeState();
-      OptionsPage.updateManagedBannerVisibility();
 
       if (this.isOverlay && !this.visible)
         OptionsPage.updateScrollPosition_();