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

chrome/browser/resources/shared/js/cr/ui/list.js

Index: chrome/browser/resources/shared/js/cr/ui/list.js
diff --git a/chrome/browser/resources/shared/js/cr/ui/list.js b/chrome/browser/resources/shared/js/cr/ui/list.js
index 588cc4b25260f655592664d543beba1a8645d3ba..0357c2f534f173fc7edb1ee9f33672b260602a9e 100644
--- a/chrome/browser/resources/shared/js/cr/ui/list.js
+++ b/chrome/browser/resources/shared/js/cr/ui/list.js
@@ -858,12 +858,6 @@ cr.define('cr.ui', function() {
 
       var autoExpands = this.autoExpands_;
       var firstIndex = autoExpands ? 0 : this.getIndexForListOffset_(scrollTop);
-      // This is a bit tricky. We take the minimum of the available items to
-      // show and the number we want to show, so as not to go off the end of the
-      // list. For the number we want to show, we take the maximum of the number
-      // that would fit without a differently-sized lead item, and with one. We
-      // do this so that if the size of the lead item changes without a scroll
-      // event to trigger redrawing the list, we won't end up with empty space.

[Ilya Sherman, 2011/09/15 04:10:39]

This comment is also included within the implementation of
getItemsInViewPort()... no need to have it twice.

       var itemsInViewPort = this.getItemsInViewPort(itemHeight, firstIndex,
           scrollTop);
       var lastIndex = firstIndex + itemsInViewPort;