diff options
| author | Svetoslav Ganov <svetoslavganov@google.com> | 2011-09-05 16:41:47 -0700 |
|---|---|---|
| committer | Svetoslav Ganov <svetoslavganov@google.com> | 2011-09-05 16:41:51 -0700 |
| commit | aa56534722a4972428e3d16b8e64caaea565a715 (patch) | |
| tree | 7a4ad3ed5d368dcc2743bf15973acb7f04c0b300 /core/java/android/widget/AdapterView.java | |
| parent | 08b997c7421735252fa927d89f22ab6ceba0bff8 (diff) | |
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
Diffstat (limited to 'core/java/android/widget/AdapterView.java')
| -rw-r--r-- | core/java/android/widget/AdapterView.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/core/java/android/widget/AdapterView.java b/core/java/android/widget/AdapterView.java index 4ba604d5adf6..b94503843f97 100644 --- a/core/java/android/widget/AdapterView.java +++ b/core/java/android/widget/AdapterView.java @@ -926,8 +926,10 @@ public abstract class AdapterView<T extends Adapter> 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 |