commit cd4f4138f05be3a1daab5035c59e97a1a82431ac
author Vadim Tryshev <vadimt@google.com> Tue May 19 14:19:37 2015 -0700
committer Vadim Tryshev <vadimt@google.com> Tue May 19 21:34:55 2015 +0000
tree 99802d1a7f9f8666e40f5b4bbb83a25f7beee9ba
parent a0abe2661eaf4f8e66990ac68386e54260836183

Removing itemCount, fromIndex and toIndex from
AccessibilityEvent.TYPE_VIEW_SCROLLED sent by PagedView.
This causes an additional reduntant voice message on scroll (see the
bug).

Also, setting these attributes violate rules set here:
http://developer.android.com/reference/android/view/accessibility/AccessibilityEvent.html
i.e. that these fields should be set only for descendants of
AdapterView.

Note that we can't just stop sending TYPE_VIEW_SCROLLED, because in this
case, accessibility focus won't be set after scrolling.

Bug: 21304383
Change-Id: I84f8e064d8209c0e09d6827551e00c9913829b57

src/com/android/launcher3/PagedView.java

diff --git a/src/com/android/launcher3/PagedView.java b/src/com/android/launcher3/PagedView.java
index c8e7d9c..e6dc59c 100644
--- a/src/com/android/launcher3/PagedView.java
+++ b/src/com/android/launcher3/PagedView.java
@@ -639,9 +639,6 @@
             if (mCurrentPage != getNextPage()) {
                 AccessibilityEvent ev =
                         AccessibilityEvent.obtain(AccessibilityEvent.TYPE_VIEW_SCROLLED);
-                ev.setItemCount(getChildCount());
-                ev.setFromIndex(getNextPage());
-                ev.setToIndex(getNextPage());
 
                 sendAccessibilityEventUnchecked(ev);
             }