From e3189a4ad7597711222589b22db1a91415641ca6 Mon Sep 17 00:00:00 2001 From: Yorke Lee Date: Wed, 1 Jul 2015 10:13:40 -0700 Subject: Don't auto-expand first voicemail card Bug: 22127956 Change-Id: Ia1a62e799bf7b535da2390bc0e8fa62201b8a684 --- src/com/android/dialer/calllog/CallLogAdapter.java | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) (limited to 'src/com/android/dialer') diff --git a/src/com/android/dialer/calllog/CallLogAdapter.java b/src/com/android/dialer/calllog/CallLogAdapter.java index 83e55944f..510847264 100644 --- a/src/com/android/dialer/calllog/CallLogAdapter.java +++ b/src/com/android/dialer/calllog/CallLogAdapter.java @@ -361,10 +361,13 @@ public class CallLogAdapter extends GroupingListAdapter @Override public void changeCursor(Cursor cursor) { - // Data has changed; reset so that the first call log list item will be expanded. - mHasBoundFirstView = false; - mCurrentlyExpandedPosition = RecyclerView.NO_POSITION; - mCurrentlyExpandedRowId = NO_EXPANDED_LIST_ITEM; + // Don't auto-expand the first item for the voicemail list fragment since that will + // trigger an unwanted voicemail download and playback. + if (mVoicemailPlaybackPresenter == null) { + mHasBoundFirstView = false; + mCurrentlyExpandedPosition = RecyclerView.NO_POSITION; + mCurrentlyExpandedRowId = NO_EXPANDED_LIST_ITEM; + } super.changeCursor(cursor); } @@ -578,8 +581,10 @@ public class CallLogAdapter extends GroupingListAdapter // In case ViewHolders were added/removed, update the expanded position if the rowIds // match so that we can restore the correct expanded state on rebind. mCurrentlyExpandedPosition = position; - } else if (!mHasBoundFirstView) { + } else if (!mHasBoundFirstView && mVoicemailPlaybackPresenter == null) { // Expand the first view when loading the call log to expose the actions. + // Don't auto-expand the first item for the voicemail list fragment since that will + // trigger an unwanted voicemail download and playback. mCurrentlyExpandedRowId = views.rowId; mCurrentlyExpandedPosition = position; mHasBoundFirstView = true; -- cgit v1.2.3