summaryrefslogtreecommitdiff
path: root/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java
diff options
context:
space:
mode:
authoruabdullah <uabdullah@google.com>2017-11-14 11:51:49 -0800
committerzachh <zachh@google.com>2017-11-18 07:25:23 +0000
commitbc9d0073e0dd5125ed215fe4d36e38354b855bd5 (patch)
tree3ce07b050dacb8fc91408858c649f5bf678f14b1 /java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java
parente8dab7ef42ba99e002db2160c6ccfff72cd5100b (diff)
Set total duration for NUI voicemail media player seekbar
The duration of the voicemail shown in the seekbar/media player (upon expanding) should be the same as the duration shown when the voicemail view holder is collapsed. Both values are retrieved from the same source (VoicemailEntry). Bug: 64882313,68382421,69268144 Test: Unit Tests PiperOrigin-RevId: 175712662 Change-Id: I70496c004d52deb1793e6d6354b0ec79bf4c4398
Diffstat (limited to 'java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java')
-rw-r--r--java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java24
1 files changed, 17 insertions, 7 deletions
diff --git a/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java b/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java
index 1e56a8189..b2d93b17c 100644
--- a/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java
+++ b/java/com/android/dialer/voicemail/listui/NewVoicemailMediaPlayerView.java
@@ -25,18 +25,19 @@ 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;
import android.view.View;
import android.widget.Button;
import android.widget.LinearLayout;
+import android.widget.TextView;
import com.android.dialer.common.Assert;
import com.android.dialer.common.LogUtil;
import com.android.dialer.common.concurrent.DialerExecutor.SuccessListener;
import com.android.dialer.common.concurrent.DialerExecutor.Worker;
import com.android.dialer.common.concurrent.DialerExecutorComponent;
+import com.android.dialer.voicemail.model.VoicemailEntry;
/**
* The view of the media player that is visible when a {@link NewVoicemailViewHolder} is expanded.
@@ -46,6 +47,7 @@ public class NewVoicemailMediaPlayerView extends LinearLayout {
private Button playButton;
private Button speakerButton;
private Button deleteButton;
+ private TextView totalDurationView;
private Uri voicemailUri;
private FragmentManager fragmentManager;
private MediaPlayer mediaPlayer;
@@ -62,14 +64,15 @@ public class NewVoicemailMediaPlayerView extends LinearLayout {
protected void onFinishInflate() {
super.onFinishInflate();
LogUtil.enterBlock("NewVoicemailMediaPlayer.onFinishInflate");
- initializeMediaPlayerButtons();
+ initializeMediaPlayerButtonsAndViews();
setupListenersForMediaPlayerButtons();
}
- private void initializeMediaPlayerButtons() {
+ private void initializeMediaPlayerButtonsAndViews() {
playButton = findViewById(R.id.playButton);
speakerButton = findViewById(R.id.speakerButton);
deleteButton = findViewById(R.id.deleteButton);
+ totalDurationView = findViewById(R.id.playback_seek_total_duration);
}
private void setupListenersForMediaPlayerButtons() {
@@ -171,7 +174,6 @@ public class NewVoicemailMediaPlayerView extends LinearLayout {
}
};
- @VisibleForTesting(otherwise = VisibleForTesting.NONE)
OnCompletionListener onCompletionListener =
new OnCompletionListener() {
@@ -197,7 +199,6 @@ public class NewVoicemailMediaPlayerView extends LinearLayout {
}
};
- @VisibleForTesting(otherwise = VisibleForTesting.NONE)
OnErrorListener onErrorListener =
new OnErrorListener() {
@Override
@@ -210,12 +211,21 @@ public class NewVoicemailMediaPlayerView extends LinearLayout {
}
};
- public void setVoicemailUri(Uri voicemailUri) {
+ void setVoicemailUri(Uri voicemailUri) {
Assert.isNotNull(voicemailUri);
this.voicemailUri = voicemailUri;
}
- public void setFragmentManager(FragmentManager fragmentManager) {
+ void setFragmentManager(FragmentManager fragmentManager) {
this.fragmentManager = fragmentManager;
}
+
+ // TODO(uabdullah): Merge with voicemailUri (http://cl/175585919)
+ void setVoicemailDuration(VoicemailEntry voicemailEntry) {
+ Assert.isNotNull(voicemailEntry);
+ Assert.isNotNull(totalDurationView);
+
+ totalDurationView.setText(
+ VoicemailEntryText.getVoicemailDuration(getContext(), voicemailEntry));
+ }
}