Fix list focus after tab key in chrome://settings/autofillEditAddress page.

chrome/browser/resources/options/autofill_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.autofillOptions', function() {
   /** @const */ var DeletableItem = options.DeletableItem;
   /** @const */ var DeletableItemList = options.DeletableItemList;
   /** @const */ var InlineEditableItem = options.InlineEditableItem;
   /** @const */ var InlineEditableItemList = options.InlineEditableItemList;
 
@@ skipping 190 matching lines @@
         return;
       }
 
       var entries = this.list.dataModel.slice();
       if (this.valueIsNonEmpty_(value) &&
           !entries.some(this.valuesAreEqual_.bind(this, value))) {
         // Update with new value.
         if (this.isPlaceholder) {
           // It is important that updateIndex is done before validateAndSave.
           // Otherwise we can not be sure about AddRow index.
+          this.list.eventsDisabled = true;
           this.list.dataModel.updateIndex(i);
+          this.list.eventsDisabled = false;
           this.list.validateAndSave(i, 0, value);
         } else {
           this.list.validateAndSave(i, 1, value);
         }
       } else {
         // Reject empty values and duplicates.
-        if (!this.isPlaceholder)
+        if (!this.isPlaceholder) {
+          this.list.eventsDisabled = true;
           this.list.dataModel.splice(i, 1);
+          this.list.eventsDisabled = false;
+          this.list.setSelectedIndexWithoutFocusing(i);
+        }

[Dan Beam, 2015/01/21 18:18:07]

} else {

[bondd, 2015/01/22 01:05:22]

Done.

         else
           this.clearValue_();
       }
     },
   };
 
   /**
    * Creates a new name value list item.
    * @param {options.autofillOptions.AutofillNameValuesList} list The parent
    *     list of this item.
@@ skipping 205 matching lines @@
     },
 
     /**
      * Called when a new list item should be validated; subclasses are
      * responsible for implementing if validation is required.
      * @param {number} index The index of the item that was inserted or changed.
      * @param {number} remove The number items to remove.
      * @param {string} value The value of the item to insert.
      */
     validateAndSave: function(index, remove, value) {
+      this.eventsDisabled = true;
       this.dataModel.splice(index, remove, value);
+      this.eventsDisabled = false;
+      this.setSelectedIndexWithoutFocusing(index);
     },
   };
 
   /**
    * Create a new value list for phone number validation.
    * @constructor
    * @extends {options.autofillOptions.AutofillValuesList}
    */
   var AutofillNameValuesList = cr.ui.define('list');
 
@@ skipping 50 matching lines @@
      * This should be called when a reply of chrome.send('validatePhoneNumbers')
      * is received.
      */
     didReceiveValidationResult: function() {
       this.validationRequests_--;
       assert(this.validationRequests_ >= 0);
       if (this.validationRequests_ <= 0) {
         while (this.validationPromiseResolvers_.length) {
           this.validationPromiseResolvers_.pop()();
         }
-        // List has been repopulated. Focus the placeholder.
-        this.focusPlaceholder();
       }
     },
 
     /**
      * Returns a Promise which is fulfilled when all of validation requests are
      * completed.
      * @return {!Promise} A promise.
      */
     doneValidating: function() {
       if (this.validationRequests_ <= 0)
@@ skipping 10 matching lines @@
     CreditCardListItem: CreditCardListItem,
     ValuesListItem: ValuesListItem,
     NameListItem: NameListItem,
     AutofillAddressList: AutofillAddressList,
     AutofillCreditCardList: AutofillCreditCardList,
     AutofillValuesList: AutofillValuesList,
     AutofillNameValuesList: AutofillNameValuesList,
     AutofillPhoneValuesList: AutofillPhoneValuesList,
   };
 });