commit 3016c1ae6a05a67106301fc5ffd7779338288e65
author Svetoslav Ganov <svetoslavganov@google.com> Mon Jun 04 19:47:40 2012 -0700
committer Svetoslav Ganov <svetoslavganov@google.com> Mon Jun 04 19:47:46 2012 -0700
tree d6e3037b8ab2e4dceb677d5f1918672fab05dd72
parent a3404be930883c420bc551d881e5f29a2e3dd2cd

NullPointerException in AbsListView.focusSearch().

1. One of the loops was using the last visible position
   as its end boundary as opposed to the child count
   minus one.

bug:6610451

Change-Id: I600545ca3f64d1e982f909f893e5d34570ec5c8b

core/java/android/widget/AbsListView.java

diff --git a/core/java/android/widget/AbsListView.java b/core/java/android/widget/AbsListView.java
index 2d207ec..5f60735 100644
--- a/core/java/android/widget/AbsListView.java
+++ b/core/java/android/widget/AbsListView.java
@@ -1388,7 +1388,7 @@
                 }
                 // Try to move focus to the next item.
                 final int nextPosition = currentPosition - getFirstVisiblePosition() + 1;
-                for (int i = nextPosition; i <= getLastVisiblePosition(); i++) {
+                for (int i = nextPosition; i < getChildCount(); i++) {
                     View child = getChildAt(i);
                     if (child.getVisibility() == View.VISIBLE) {
                         return child;