From 47be54bb789cae27eaba83890df4596cce1dee09 Mon Sep 17 00:00:00 2001 From: Alan Viverette Date: Fri, 5 Aug 2016 16:48:19 -0400 Subject: [PATCH] Use symmetric conditions for advertising and performing ACTION_SCROLL Bug: 30186372 Change-Id: Ieea0cd539049990ffa98edabb2a7b026445dd14a --- core/java/android/widget/AbsListView.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/core/java/android/widget/AbsListView.java b/core/java/android/widget/AbsListView.java index b65f93309e048..7b45d8cb05b09 100644 --- a/core/java/android/widget/AbsListView.java +++ b/core/java/android/widget/AbsListView.java @@ -1546,7 +1546,7 @@ public abstract class AbsListView extends AdapterView implements Te switch (action) { case AccessibilityNodeInfo.ACTION_SCROLL_FORWARD: case R.id.accessibilityActionScrollDown: { - if (isEnabled() && getLastVisiblePosition() < getCount() - 1) { + if (isEnabled() && canScrollDown()) { final int viewportHeight = getHeight() - mListPadding.top - mListPadding.bottom; smoothScrollBy(viewportHeight, PositionScroller.SCROLL_DURATION); return true; @@ -1554,7 +1554,7 @@ public abstract class AbsListView extends AdapterView implements Te } return false; case AccessibilityNodeInfo.ACTION_SCROLL_BACKWARD: case R.id.accessibilityActionScrollUp: { - if (isEnabled() && mFirstPosition > 0) { + if (isEnabled() && canScrollUp()) { final int viewportHeight = getHeight() - mListPadding.top - mListPadding.bottom; smoothScrollBy(-viewportHeight, PositionScroller.SCROLL_DURATION); return true;