Cherry pick IndexListAdapter fix into i-dev (2d60f950) · Commits · e / os / android_packages_apps_Contacts

src/com/android/contacts/list/IndexerListAdapter.java

+21 −1

Original line number	Diff line number	Diff line
		@@ -173,7 +173,7 @@ public abstract class IndexerListAdapter extends PinnedHeaderListAdapter impleme
		if (section == -1) {
		listView.setHeaderInvisible(index, false);
		} else {
		View topChild = listView.getChildAt(listPosition);
		View topChild = getViewAtVisiblePosition(listView, listPosition);
		if (topChild != null) {
		// Match the pinned header's height to the height of the list item.
		mHeader.setMinimumHeight(topChild.getMeasuredHeight());
		@@ -194,6 +194,26 @@ public abstract class IndexerListAdapter extends PinnedHeaderListAdapter impleme
		}
		}

		/**
		* Returns the view used for the specified list position assuming it is visible or null if
		* it isn't.
		*
		* <p>This makes some assumptions about the implementation of ListView (child views are the
		* item views and are ordered in the same way as the adapter items they are displaying)
		* but they are probably safe given that the API is stable.</p>
		*/
		private View getViewAtVisiblePosition(ListView list, int position) {
		final int firstVisiblePosition = list.getFirstVisiblePosition();
		final int childCount = list.getChildCount();
		final int index = position - firstVisiblePosition;
		if (index >= 0 && index < childCount) {
		// Position is on-screen, use existing view.
		return list.getChildAt(index);
		} else {
		return null;
		}
		}

		/**
		* Computes the item's placement within its section and populates the {@code placement}
		* object accordingly. Please note that the returned object is volatile and should be