OSDN Git Service

Merge "DO NOT MERGE AdapterView does not set setCurrentItemIndex in accessibility...
authorSvetoslav Ganov <svetoslavganov@google.com>
Tue, 11 Oct 2011 20:54:28 +0000 (13:54 -0700)
committerAndroid (Google) Code Review <android-gerrit@google.com>
Tue, 11 Oct 2011 20:54:28 +0000 (13:54 -0700)
1  2 
core/java/android/widget/AdapterView.java

@@@ -922,19 -922,16 +922,20 @@@ public abstract class AdapterView<T ext
          if (selectedView != null) {
              event.setEnabled(selectedView.isEnabled());
          }
+         event.setCurrentItemIndex(getSelectedItemPosition());
          event.setFromIndex(getFirstVisiblePosition());
          event.setToIndex(getLastVisiblePosition());
          event.setItemCount(getAdapter().getCount());
      }
  
      private boolean isScrollableForAccessibility() {
 -        final int itemCount = getAdapter().getCount();
 -        return itemCount > 0
 -            && (getFirstVisiblePosition() > 0 || getLastVisiblePosition() < itemCount - 1);
 +        T adapter = getAdapter();
 +        if (adapter != null) {
 +            final int itemCount = adapter.getCount();
 +            return itemCount > 0
 +                && (getFirstVisiblePosition() > 0 || getLastVisiblePosition() < itemCount - 1);
 +        }
 +        return false;
      }
  
      @Override