Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1726)

Unified Diff: chrome/browser/resources/shared/js/cr/ui/list.js

Issue 7903001: Improve Tab-key navigation for editing Autofill addresses and credit cards. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: De-nitting Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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.
var itemsInViewPort = this.getItemsInViewPort(itemHeight, firstIndex,
scrollTop);
var lastIndex = firstIndex + itemsInViewPort;

Powered by Google App Engine
This is Rietveld 408576698