Improve Tab-key navigation for editing Autofill addresses and credit cards.

chrome/browser/resources/options/inline_editable_list.js

Index: chrome/browser/resources/options/inline_editable_list.js
diff --git a/chrome/browser/resources/options/inline_editable_list.js b/chrome/browser/resources/options/inline_editable_list.js
index 6cd088667f92dfe1c623efc5e869003d7b01e92e..8aed93beedf423fe5d24ca8c6f317fa50979672e 100644
--- a/chrome/browser/resources/options/inline_editable_list.js
+++ b/chrome/browser/resources/options/inline_editable_list.js
@@ -83,18 +83,6 @@ cr.define('options', function() {
     },
 
     /**
-     * Called when the input element receives focus. Selects this item in the
-     * list selection model.
-     * @private
-     */
-    handleFocus_: function() {
-      var list = this.parentNode;
-      var index = list.getIndexOfListItem(this);
-      list.selectionModel.selectedIndex = index;
-      list.selectionModel.anchorIndex = index;
-    },
-
-    /**
      * Called when this element gains or loses 'lead' status. Updates editing
      * mode accordingly.
      * @private
@@ -142,7 +130,7 @@ cr.define('options', function() {
         if (focusElement) {
           window.setTimeout(function() {
             // Make sure we are still in edit mode by the time we execute.
-            if (self.editing && self.focusPlaceholder) {

[James Hawkins, 2011/09/19 23:00:07]

You're right. I checked it out, and this check is wrong and unnecessary if it
were modified to be correct (self.parent.focusPlaceholder).

+            if (self.editing) {
               focusElement.focus();
               focusElement.select();
             }