diff options
author | Andrew Lee <anwlee@google.com> | 2015-06-20 01:18:51 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2015-06-20 01:18:52 +0000 |
commit | f3ec8ad515ec344acd53fa4acaa9df4cad0c2306 (patch) | |
tree | 557ead47e7e583cb8b37b8c4a9cb9cf664d0ae15 /src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java | |
parent | 03aaf3894d8af53f241d1189c504e44c538100d4 (diff) | |
parent | 053b9c7a4b2662588706211fd8aa1f8a03ec2901 (diff) |
Merge "Rewrite of MediaPlayer logic." into mnc-dev
Diffstat (limited to 'src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java')
-rw-r--r-- | src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java index 73f4b3b1c..ca487db56 100644 --- a/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java +++ b/src/com/android/dialer/voicemail/VoicemailPlaybackLayout.java @@ -50,8 +50,9 @@ import javax.annotation.concurrent.NotThreadSafe; import javax.annotation.concurrent.ThreadSafe; /** - * Displays and plays a single voicemail. - * <p> + * Displays and plays a single voicemail. See {@link VoicemailPlaybackPresenter} for + * details on the voicemail playback implementation. + * * This class is not thread-safe, it is thread-confined. All calls to all public * methods on this class are expected to come from the main ui thread. */ @@ -178,12 +179,13 @@ public class VoicemailPlaybackLayout extends LinearLayout if (mPresenter == null) { return; } - CallLogAsyncTaskUtil.deleteVoicemail(mContext, mPresenter.getVoicemailUri(), null); + CallLogAsyncTaskUtil.deleteVoicemail(mContext, mVoicemailUri, null); } }; private Context mContext; private VoicemailPlaybackPresenter mPresenter; + private Uri mVoicemailUri; private boolean mIsPlaying = false; @@ -209,8 +211,9 @@ public class VoicemailPlaybackLayout extends LinearLayout } @Override - public void setPresenter(VoicemailPlaybackPresenter presenter) { + public void setPresenter(VoicemailPlaybackPresenter presenter, Uri voicemailUri) { mPresenter = presenter; + mVoicemailUri = voicemailUri; } @Override @@ -256,15 +259,13 @@ public class VoicemailPlaybackLayout extends LinearLayout } @Override - public void onPlaybackError(Exception e) { + public void onPlaybackError() { if (mPositionUpdater != null) { mPositionUpdater.stopUpdating(); } disableUiElements(); mPlaybackPosition.setText(getString(R.string.voicemail_playback_error)); - - Log.e(TAG, "Could not play voicemail", e); } |