From f35cf6c6311aa253d679f61b8fd43babce4a90c3 Mon Sep 17 00:00:00 2001
From: Ta-wei Yen <twyen@google.com>
Date: Mon, 21 Mar 2016 20:35:26 -0700
Subject: 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
---
 src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java    | 5 +++++
 src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java | 2 ++
 2 files changed, 7 insertions(+)

(limited to 'src/com')

diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java
index d4d294e8d..7d6fe78d1 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
         mTotalDurationText.setText(formatAsMinutesAndSeconds(durationMs));
     }
 
+    @Override
+    public void setSuccess() {
+        mStateText.setText(null);
+    }
+
     @Override
     public void setIsFetchingContent() {
         disableUiElements();
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) {
-- 
cgit v1.2.3