Fewer focusable items in chrome://settings/searchEngines

chrome/browser/resources/options/deletable_item_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 List = cr.ui.List;
   /** @const */ var ListItem = cr.ui.ListItem;
 
   /**
    * Creates a deletable list item, which has a button that will trigger a call
@@ skipping 20 matching lines @@
      */
     closeButtonElement_: null,
 
     /**
      * Whether or not this item can be deleted.
      * @type {boolean}
      * @private
      */
     deletable_: true,
 
+    /**
+     * Whether or not the close button can ever be navigated to using the
+     * keyboard.
+     * @type {boolean}

[Dan Beam, 2014/10/29 01:37:12]

@protected

[bondd, 2014/10/29 17:19:07]

Done.

+     */
+    closeButtonFocusAllowed: false,
+
     /** @override */
     decorate: function() {
       ListItem.prototype.decorate.call(this);
 
       this.classList.add('deletable-item');
 
       this.contentElement_ = /** @type {HTMLElement} */(
           this.ownerDocument.createElement('div'));
       this.appendChild(this.contentElement_);
 
       this.closeButtonElement_ = /** @type {HTMLElement} */(
           this.ownerDocument.createElement('button'));
       this.closeButtonElement_.className =
           'raw-button row-delete-button custom-appearance';
       this.closeButtonElement_.addEventListener('mousedown',
                                                 this.handleMouseDownUpOnClose_);
       this.closeButtonElement_.addEventListener('mouseup',
                                                 this.handleMouseDownUpOnClose_);
       this.closeButtonElement_.addEventListener('focus',
-                                                this.handleFocus_.bind(this));
+                                                this.handleFocus.bind(this));
       this.closeButtonElement_.tabIndex = -1;
       this.closeButtonElement_.title =
           loadTimeData.getString('deletableItemDeleteButtonTitle');
       this.appendChild(this.closeButtonElement_);
     },
 
     /**
      * Returns the element subclasses should add content to.
      * @return {HTMLElement} The element subclasses should popuplate.
      */
@@ skipping 16 matching lines @@
       return this.deletable_;
     },
     set deletable(value) {
       this.deletable_ = value;
       this.closeButtonElement_.disabled = !value;
     },
 
     /**
      * Called when a focusable child element receives focus. Selects this item
      * in the list selection model.
-     * @private
+     * @protected
      */
-    handleFocus_: function() {
+    handleFocus: function() {

[bondd, 2014/10/28 22:29:57]

Made this protected because existing code in inline_editable_list.js was already
using it that way.

       // This handler is also fired when the child receives focus as a result of
       // the item getting selected by the customized mouse/keyboard handling in
       // SelectionController. Take care not to destroy a potential multiple
       // selection in this case.
       if (this.selected)
         return;
 
       var list = this.parentNode;
       var index = list.getIndexOfListItem(this);
       list.selectionModel.selectedIndex = index;
@@ skipping 87 matching lines @@
      */
     deleteItemAtIndex: function(index) {
     },
   };
 
   return {
     DeletableItemList: DeletableItemList,
     DeletableItem: DeletableItem,
   };
 });