Add policy indicator for protocol handlers

chrome/browser/resources/options/handler_options_list.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', function() {
   /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
   /** @const */ var List = cr.ui.List;
   /** @const */ var ListItem = cr.ui.ListItem;
   /** @const */ var DeletableItem = options.DeletableItem;
   /** @const */ var DeletableItemList = options.DeletableItemList;
@@ skipping 133 matching lines @@
           handlerElement.classList.remove('none');
           delegate.setDefault(data.handlers[index]);
         }
       });
       handlerElement.appendChild(selectElement);
       handlerElement.className = 'handlers-site-column';
       if (data.default_handler == -1)
         this.classList.add('none');
       this.appendChild(handlerElement);
 
       // Remove link.

[bartfab (slow), 2014/09/01 09:38:37]

Nit: Move the comment inside the conditional. It applies to the first branch
only.

-      var removeElement = document.createElement('div');
-      removeElement.textContent =
-          loadTimeData.getString('handlers_remove_link');
-      removeElement.addEventListener('click', function(e) {
-        var value = selectElement ? selectElement.value : 0;
-        delegate.removeHandler(value, data.handlers[value]);
-      });
-      removeElement.className = 'handlers-remove-column handlers-remove-link';
-      this.appendChild(removeElement);
+      if (!data.policy_controlled) {
+        var removeElement = document.createElement('div');
+        removeElement.textContent =
+            loadTimeData.getString('handlers_remove_link');
+        removeElement.addEventListener('click', function(e) {
+          var value = selectElement ? selectElement.value : 0;
+          delegate.removeHandler(value, data.handlers[value]);
+        });
+        removeElement.className =
+            'handlers-remove-column handlers-remove-link';
+        this.appendChild(removeElement);
+      } else if (data.default_handler != -1) {
+        var indicator = new options.ControlledSettingIndicator();

[bartfab (slow), 2014/09/01 09:38:37]

Nit: Add a comment that explains this is a policy indicator.

+        indicator.controlledBy = 'hasRecommendation';
+        this.appendChild(indicator);
+      }
     },
 
     /** @override */
     decorate: function() {
       ListItem.prototype.decorate.call(this);
 
       var delegate = {
         removeHandler: function(index, handler) {
           chrome.send('removeHandler', [handler]);
         },
@@ skipping 42 matching lines @@
     },
   };
 
   return {
     IgnoredHandlersListItem: IgnoredHandlersListItem,
     IgnoredHandlersList: IgnoredHandlersList,
     HandlerListItem: HandlerListItem,
     HandlersList: HandlersList,
   };
 });