diff options
author | uabdullah <uabdullah@google.com> | 2017-11-14 12:28:52 -0800 |
---|---|---|
committer | zachh <zachh@google.com> | 2017-11-18 07:25:39 +0000 |
commit | c7b413058c765603d5c65d39e4042a55ab170ef0 (patch) | |
tree | 158dbcfcb05e48dfd3f04799ea2fede919cab7ec /java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java | |
parent | bc9d0073e0dd5125ed215fe4d36e38354b855bd5 (diff) |
Combine voicemail uri and duration for NUI media player
Voicemail URI and duration are both retrieved from VoicemailEntry. This CL combines it.
This was a todo added from http://cl/175585919
Test: unit tests
PiperOrigin-RevId: 175718355
Change-Id: I7d2748cf6e110e494f6895c512ce355aa2d048a7
Diffstat (limited to 'java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java')
-rw-r--r-- | java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java b/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java index b2d93b17c..4629ce277 100644 --- a/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java +++ b/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java @@ -25,6 +25,7 @@ import android.media.MediaPlayer.OnErrorListener; import android.media.MediaPlayer.OnPreparedListener; import android.net.Uri; import android.provider.VoicemailContract; +import android.support.annotation.VisibleForTesting; import android.support.v4.util.Pair; import android.util.AttributeSet; import android.view.LayoutInflater; @@ -174,6 +175,7 @@ public class NewVoicemailMediaPlayerView extends LinearLayout { } }; + @VisibleForTesting(otherwise = VisibleForTesting.PRIVATE) OnCompletionListener onCompletionListener = new OnCompletionListener() { @@ -199,6 +201,7 @@ public class NewVoicemailMediaPlayerView extends LinearLayout { } }; + @VisibleForTesting(otherwise = VisibleForTesting.PRIVATE) OnErrorListener onErrorListener = new OnErrorListener() { @Override @@ -211,20 +214,17 @@ public class NewVoicemailMediaPlayerView extends LinearLayout { } }; - void setVoicemailUri(Uri voicemailUri) { - Assert.isNotNull(voicemailUri); - this.voicemailUri = voicemailUri; - } - void setFragmentManager(FragmentManager fragmentManager) { this.fragmentManager = fragmentManager; } - // TODO(uabdullah): Merge with voicemailUri (http://cl/175585919) - void setVoicemailDuration(VoicemailEntry voicemailEntry) { + void setVoicemailEntryValues(VoicemailEntry voicemailEntry) { Assert.isNotNull(voicemailEntry); + Uri uri = Uri.parse(voicemailEntry.voicemailUri()); + Assert.isNotNull(uri); Assert.isNotNull(totalDurationView); + voicemailUri = uri; totalDurationView.setText( VoicemailEntryText.getVoicemailDuration(getContext(), voicemailEntry)); } |