Revert of [Chrome Settings UI] If User Exceptions are not allowed, prevent editing / viewing.

chrome/browser/resources/options/content_settings_exceptions_area.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('options.contentSettings', function() {
   /** @const */ var ControlledSettingIndicator =
                     options.ControlledSettingIndicator;
   /** @const */ var InlineEditableItemList = options.InlineEditableItemList;
   /** @const */ var InlineEditableItem = options.InlineEditableItem;
   /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
@@ skipping 460 matching lines @@
    *
    * @constructor
    * @extends {options.InlineEditableItemList}
    */
   var ExceptionsList = cr.ui.define('list');
 
   ExceptionsList.prototype = {
     __proto__: InlineEditableItemList.prototype,
 
     /**
-     * True if user may set preference exceptions.
-     * @private {boolean}
-     */
-    allowEdit_: true,
-
-    /**
      * Called when an element is decorated as a list.
      */
     decorate: function() {
       InlineEditableItemList.prototype.decorate.call(this);
 
       this.classList.add('settings-list');
 
       for (var parentNode = this.parentNode; parentNode;
            parentNode = parentNode.parentNode) {
         if (parentNode.hasAttribute('contentType')) {
@@ skipping 22 matching lines @@
                                       entry);
       } else {
         var addRowItem = new ExceptionsAddRowListItem(this.contentType,
                                                       this.mode);
         addRowItem.deletable = false;
         return addRowItem;
       }
     },
 
     /**
-     * Updates visibility of each row, depending on content.
-     */
-    updateRowVisibility: function() {
-      // Rows with user exceptions are visible iff this.isEditable().
-      // All other rows are visible.
-      var isEditable = this.isEditable();
-      for (var index = 0; index < this.dataModel.length; ++index) {
-        var item = this.getListItemByIndex(index);
-        if (item.dataItem.source == 'preference')
-          item.hidden = !isEditable;
-      }
-    },
-
-    /**
-     * Sets whether user is allowed to set preference exceptions. Updates UI.
-     *
-     * @param {boolean} allowEdit New value for this.allowEdit_.
-     */
-    setAllowEdit: function(allowEdit) {
-      var oldIsEditable = this.isEditable();
-      this.allowEdit_ = allowEdit;
-      var newIsEditable = this.isEditable();
-
-      // If visibility changed, add or remove the Add New Exception row.
-      if (oldIsEditable != newIsEditable) {
-        if (newIsEditable)
-          this.dataModel.push(null);
-        else
-          this.dataModel.pop();
-        this.updateRowVisibility();
-      }
-    },
-
-    /**
      * Sets the exceptions in the js model.
      *
      * @param {Array<options.Exception>} entries A list of dictionaries of
      *     values, each dictionary represents an exception.
      */
     setExceptions: function(entries) {
       var deleteCount = this.dataModel.length;
 
       if (this.isEditable()) {
         // We don't want to remove the Add New Exception row.
         deleteCount = deleteCount - 1;
       }
 
       var args = [0, deleteCount];
       args.push.apply(args, entries);
       this.dataModel.splice.apply(this.dataModel, args);
-      this.updateRowVisibility();
     },
 
     /**
      * The browser has finished checking a pattern for validity. Update the list
      * item to reflect this.
      *
      * @param {string} pattern The pattern.
      * @param {boolean} valid Whether said pattern is valid in the context of a
      *     content exception setting.
      */
     patternValidityCheckComplete: function(pattern, valid) {
       var listItems = this.items;
       for (var i = 0; i < listItems.length; i++) {
         var listItem = listItems[i];
         // Don't do anything for messages for the item if it is not the intended
         // recipient, or if the response is stale (i.e. the input value has
         // changed since we sent the request to analyze it).
         if (pattern == listItem.input.value)
           listItem.setPatternValid(valid);
       }
     },
 
     /**
      * Returns whether the rows are editable in this list.
      */
     isEditable: function() {
-      return this.allowEdit_ && isEditableType(this.contentType);
+      // Exceptions of the following lists are not editable for now.
+      return isEditableType(this.contentType);
     },
 
     /**
      * Removes all exceptions from the js model.
      */
     reset: function() {
-      this.allowEdit_ = true;
       if (this.isEditable()) {
         // The null creates the Add New Exception row.
         this.dataModel = new ArrayDataModel([null]);
       } else {
         this.dataModel = new ArrayDataModel([]);
       }
     },
 
     /** @override */
     deleteItemAtIndex: function(index) {
@@ skipping 113 matching lines @@
     }
   };
 
   return {
     ExceptionsListItem: ExceptionsListItem,
     ExceptionsAddRowListItem: ExceptionsAddRowListItem,
     ExceptionsList: ExceptionsList,
     ContentSettingsExceptionsArea: ContentSettingsExceptionsArea,
   };
 });