commit f4b1101e5831faa5407e44566185007788090dce
author Jay Shrauner <shrauner@google.com> Mon Oct 12 16:34:56 2015 -0700
committer Yorke Lee <yorkelee@google.com> Tue Oct 20 09:38:59 2015 -0700
tree 32655a290df611143bab65ec89281553af67c9ac
parent ccf6c7a8b3da1da6bf9df41e18485ddc08a5a17a

Fix NPE in requestContent

Check for null context in resumePlayback.

Bug:24858686
Change-Id: I7c65e3ae836535f15936ecb397a68ee6590f31ed

src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java

diff --git a/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java b/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java
index 9319b6e..ed6cc8b 100644
--- a/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java
+++ b/src/com/android/dialer/voicemail/VoicemailPlaybackPresenter.java
@@ -587,6 +587,10 @@
      * playing.
      */
     public void resumePlayback() {
+        if (mView == null || mContext == null) {
+            return;
+        }
+
         if (!mIsPrepared) {
             // If we haven't downloaded the voicemail yet, attempt to download it.
             checkForContent();
@@ -597,7 +601,7 @@
 
         mIsPlaying = true;
 
-        if (!mMediaPlayer.isPlaying()) {
+        if (mMediaPlayer != null && !mMediaPlayer.isPlaying()) {
             // Clamp the start position between 0 and the duration.
             mPosition = Math.max(0, Math.min(mPosition, mDuration.get()));
             mMediaPlayer.seekTo(mPosition);