Add controlled setting indicators for content settings

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;
 
   /**
    * Creates a new exceptions list item.
    *
    * @param {string} contentType The type of the list.
    * @param {string} mode The browser mode, 'otr' or 'normal'.
    * @param {boolean} enableAskOption Whether to show an 'ask every time'
@@ skipping 26 matching lines @@
 
       this.isPlaceholder = !this.pattern;
       var patternCell = this.createEditableTextCell(this.pattern);
       patternCell.className = 'exception-pattern';
       patternCell.classList.add('weakrtl');
       this.contentElement.appendChild(patternCell);
       if (this.pattern)
         this.patternLabel = patternCell.querySelector('.static-text');
       var input = patternCell.querySelector('input');
 
+      // If the source of the content setting exception is not a user
+      // preference, that source controls the exception and the user cannot edit
+      // or delete it.
+      var controlledBy =
+          this.dataItem.source && this.dataItem.source != 'preference' ?
+              this.dataItem.source : null;

[Joao da Silva, 2012/10/15 08:02:32]

How about:

var controlledBy = this.dataItem.source === 'preference' ? null :
this.dataItem.source;

Also, move this next to the place where it's used.

[bartfab (slow), 2012/10/15 12:34:07]

Move done. The change to the code you suggested will not work though: Checking
for 'preference' is not enough. this.dataItem.source may be undefined as well.

[Joao da Silva, 2012/10/15 12:53:59]

|controlledBy| is always tested before begin used here, so having it undefined
or null would be the same. Minor detail, do what you prefer :-)

+
       // TODO(stuartmorgan): Create an createEditableSelectCell abstracting
       // this code.
       // Setting label for display mode. |pattern| will be null for the 'add new
       // exception' row.
       if (this.pattern) {
         var settingLabel = cr.doc.createElement('span');
         settingLabel.textContent = this.settingForDisplay();
         settingLabel.className = 'exception-setting';
         settingLabel.setAttribute('displaymode', 'static');
         this.contentElement.appendChild(settingLabel);
@@ skipping 63 matching lines @@
       this.updateEditables();
 
       // Editing notifications, geolocation and media-stream is disabled for
       // now.
       if (this.contentType == 'notifications' ||
           this.contentType == 'location' ||
           this.contentType == 'media-stream') {
         this.editable = false;
       }
 
-      // If the source of the content setting exception is not the user
-      // preference, then the content settings exception is managed and the user
-      // can't edit it.
-      if (this.dataItem.source &&
-          this.dataItem.source != 'preference') {
-        this.setAttribute('managedby', this.dataItem.source);
+      if (controlledBy) {
+        this.setAttribute('controlled-by', controlledBy);
         this.deletable = false;
         this.editable = false;
       }
 
+      if (controlledBy == 'policy' || controlledBy == 'extension') {
+        this.querySelector('.row-delete-button').hidden = true;
+        var indicator = ControlledSettingIndicator();
+        indicator.setAttribute('content-exception', this.contentType);
+        // Create a synthetic pref change event decorated as
+        // CoreOptionsHandler::CreateValueForPref() does.
+        var event = new cr.Event(this.contentType);
+        event.value = { controlledBy: controlledBy };
+        indicator.handlePrefChange(event);
+        this.appendChild(indicator);
+      }
+
       // If the exception comes from a hosted app, display the name and the
       // icon of the app.
-      if (this.dataItem.source == 'HostedApp') {
+      if (controlledBy == 'HostedApp') {
         this.title =
             loadTimeData.getString('set_by') + ' ' + this.dataItem.appName;
         var button = this.querySelector('.row-delete-button');
         // Use the host app's favicon (16px, match bigger size).
         // See c/b/ui/webui/extensions/extension_icon_source.h
         // for a description of the chrome://extension-icon URL.
         button.style.backgroundImage =
             'url(\'chrome://extension-icon/' + this.dataItem.appId + '/16/1\')';
       }
 
@@ skipping 451 matching lines @@
     }
   };
 
   return {
     ExceptionsListItem: ExceptionsListItem,
     ExceptionsAddRowListItem: ExceptionsAddRowListItem,
     ExceptionsList: ExceptionsList,
     ContentSettingsExceptionsArea: ContentSettingsExceptionsArea,
   };
 });