From a119bca4948fcf0e8289f137f402417ea18caedb Mon Sep 17 00:00:00 2001 From: Andrew Lee Date: Wed, 26 Aug 2015 16:27:12 -0700 Subject: Populate TextViews with 00:00 after inflation. But, set them to INVISIBLE, so they layout, but don't appear. This prevents jank caused by the seekbar resizing later on when times are set for the first time. Bug: 22333980 Change-Id: I8523659b837008bae1789c1beaccc7edf9c15554 --- src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java | 8 ++++++++ src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) (limited to 'src/com/android/dialer/voicemail') diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java index 158ed5834..755460114 100644 --- a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java +++ b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java @@ -240,6 +240,11 @@ public class VoicemailPlaybackLayout extends LinearLayout mStartStopButton.setOnClickListener(mStartStopButtonListener); mPlaybackSpeakerphone.setOnClickListener(mSpeakerphoneListener); mDeleteButton.setOnClickListener(mDeleteButtonListener); + + mPositionText.setText(formatAsMinutesAndSeconds(0)); + mPositionText.setVisibility(View.INVISIBLE); + mTotalDurationText.setText(formatAsMinutesAndSeconds(0)); + mTotalDurationText.setVisibility(View.INVISIBLE); } @Override @@ -355,6 +360,9 @@ public class VoicemailPlaybackLayout extends LinearLayout mStartStopButton.setEnabled(true); mPlaybackSpeakerphone.setEnabled(true); mPlaybackSeek.setEnabled(true); + + mPositionText.setVisibility(View.VISIBLE); + mTotalDurationText.setVisibility(View.VISIBLE); } private String getString(int resId) { diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java b/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java index 7270af787..056dcc64e 100644 --- a/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java +++ b/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java @@ -515,9 +515,9 @@ public class VoicemailPlaybackPresenter mDuration.set(mMediaPlayer.getDuration()); mPosition = mMediaPlayer.getCurrentPosition(); - mView.enableUiElements(); Log.d(TAG, "onPrepared: mPosition=" + mPosition); mView.setClipPosition(mPosition, mDuration.get()); + mView.enableUiElements(); mMediaPlayer.seekTo(mPosition); if (mIsPlaying) { -- cgit v1.2.3