commit cebedcf0d2b05e8f7d68ac20f724de7ae3e1327c
author Makoto Onuki <omakoto@google.com> Fri Aug 26 14:55:17 2011 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Fri Aug 26 14:55:17 2011 -0700
tree 76d0ed3c9b1df498aa423dea7ffc7d8a12bb7c47
parent 73b480880d7a7866ccae52b13018f8fb3e832db0
parent 44b90b09a61c53cd61b6f8ff0889f85b7d3490bb

Merge "Fix group list scroll position resetting issue"

src/com/android/contacts/group/GroupBrowseListFragment.java

diff --git a/src/com/android/contacts/group/GroupBrowseListFragment.java b/src/com/android/contacts/group/GroupBrowseListFragment.java
index 2b9c594..82539e8 100644
--- a/src/com/android/contacts/group/GroupBrowseListFragment.java
+++ b/src/com/android/contacts/group/GroupBrowseListFragment.java
@@ -220,12 +220,9 @@
         }
         mAdapter.setCursor(mGroupListCursor);
 
-        Parcelable listState = mListView.onSaveInstanceState();
         if (mSelectionToScreenRequested) {
+            mSelectionToScreenRequested = false;
             requestSelectionToScreen();
-        } else {
-            // Restore the scroll position.
-            mListView.onRestoreInstanceState(listState);
         }
 
         mSelectedGroupUri = mAdapter.getSelectedGroup();
@@ -262,6 +259,9 @@
     }
 
     protected void requestSelectionToScreen() {
+        if (!mSelectionVisible) {
+            return; // If selection isn't visible we don't care.
+        }
         int selectedPosition = mAdapter.getSelectedGroupPosition();
         if (selectedPosition != -1) {
             mListView.requestPositionToScreen(selectedPosition,