From aa56534722a4972428e3d16b8e64caaea565a715 Mon Sep 17 00:00:00 2001 From: Svetoslav Ganov Date: Mon, 5 Sep 2011 16:41:47 -0700 Subject: [PATCH] AbsListView does not populate accessibility events correctly. The toIndex of accessibility events fired from a AbsListView is exclusive but should be inclusive i.e. it was reported one more that it has to be. bug:5256286 Change-Id: I496959fdfb6760b0c74899730c4cc558e89234a6 --- core/java/android/widget/AdapterView.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/core/java/android/widget/AdapterView.java b/core/java/android/widget/AdapterView.java index 4ba604d5adf6e..b94503843f974 100644 --- a/core/java/android/widget/AdapterView.java +++ b/core/java/android/widget/AdapterView.java @@ -926,8 +926,10 @@ public abstract class AdapterView extends ViewGroup { } event.setItemCount(getCount()); event.setCurrentItemIndex(getSelectedItemPosition()); - event.setFromIndex(mFirstPosition); - event.setToIndex(mFirstPosition + getChildCount()); + if (getChildCount() > 0) { + event.setFromIndex(getFirstVisiblePosition()); + event.setToIndex(getLastVisiblePosition()); + } } @Override