Improve keyboard navigation on 'Manage search engines' options page

chrome/browser/resources/options/autofill_options_list.js

Index: chrome/browser/resources/options/autofill_options_list.js
diff --git a/chrome/browser/resources/options/autofill_options_list.js b/chrome/browser/resources/options/autofill_options_list.js
index 565a62e6d353c7733f663f2c32fdc5792ab4b3e8..297bd2ae0f748bdf9345e18cfa6d78b4a0699390 100644
--- a/chrome/browser/resources/options/autofill_options_list.js
+++ b/chrome/browser/resources/options/autofill_options_list.js
@@ -63,6 +63,8 @@ cr.define('options.autofillOptions', function() {
         this.guid,
         AutofillOptions.loadAddressEditor);
       this.contentElement.appendChild(editButtonEl);
+
+      this.closeButtonFocusable = true;

[Dan Beam, 2014/10/18 01:35:43]

what's the benefit to this very specific accessor over just using the current
|closeButtonElement| like this:

  this.closeButtonElement.tabIndex = 0;

or

  this.closeButtonElement.removeAttribute('tabindex');

?  tabIndex will always be public (unless we attempt to change this), so it
seems odd to try to hide it otherwise...

     },
   };
 
@@ -108,6 +110,8 @@ cr.define('options.autofillOptions', function() {
         this.guid,
         AutofillOptions.loadCreditCardEditor);
       this.contentElement.appendChild(editButtonEl);
+
+      this.closeButtonFocusable = true;
     },
   };