diff options
author | Ta-wei Yen <twyen@google.com> | 2016-03-21 20:35:26 -0700 |
---|---|---|
committer | Ta-wei Yen <twyen@google.com> | 2016-03-22 18:57:18 +0000 |
commit | 47af91907bc39b7b94173cc14809b932f516b1a2 (patch) | |
tree | c1f8d7015146426166657e6311725d6daf54fd4c | |
parent | 5e530f2d469cae13347a666fd9fe9d214d9e30b4 (diff) |
Clear voicemail status text when loading succeeded
In ag/874775, mStateText.setText(null) is removed from
VoicemailPlaybackLayout#setClipPosition(). This caused the error in
mStateText not to be cleared after the voicemail is sucessfully loaded.
This CL adds a setSuccess() method to the layout that clears the message,
which will be called from onPrepared() once the voicemail is loaded.
Fixes:27770842
Change-Id: Ic42e40b99d36f817532dec780b63311c1967300c
(cherry picked from commit f35cf6c6311aa253d679f61b8fd43babce4a90c3)
-rw-r--r-- | src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java | 5 | ||||
-rw-r--r-- | src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java | 2 |
2 files changed, 7 insertions, 0 deletions
diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java index e5efbeca3..521671a13 100644 --- a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java +++ b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java @@ -427,6 +427,11 @@ public class VoicemailPlaybackLayout extends LinearLayout } @Override + public void setSuccess() { + mStateText.setText(null); + } + + @Override public void setIsFetchingContent() { disableUiElements(); mStateText.setText(getString(R.string.voicemail_fetching_content)); diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java b/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java index 5924fb453..e224ddc2a 100644 --- a/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java +++ b/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java @@ -84,6 +84,7 @@ public class VoicemailPlaybackPresenter implements MediaPlayer.OnPreparedListene void onPlaybackStopped(); void onSpeakerphoneOn(boolean on); void setClipPosition(int clipPositionInMillis, int clipLengthInMillis); + void setSuccess(); void setFetchContentTimeout(); void setIsFetchingContent(); void onVoicemailArchiveSucceded(Uri voicemailUri); @@ -606,6 +607,7 @@ public class VoicemailPlaybackPresenter implements MediaPlayer.OnPreparedListene Log.d(TAG, "onPrepared: mPosition=" + mPosition); mView.setClipPosition(mPosition, mDuration.get()); mView.enableUiElements(); + mView.setSuccess(); mMediaPlayer.seekTo(mPosition); if (mIsPlaying) { |