commit 8298b47f0081294cb646852573bf50273440f98c
author Udam Saini <udam@google.com> Tue Jan 12 14:43:43 2016 -0800
committer Udam Saini <udam@google.com> Tue Jan 12 14:43:43 2016 -0800
tree e951e6722a66591e4dc842f70b715f38ac68cf5f
parent bd99c438874b848f7f2a2f821fb65eb63c380a00

Fixes animation video from not playing after onStop

Adds a pause command that doesn't destroy the video surface, and calls
onStop when we want to destroy the media player.

bug:26516460
Change-Id: If46d26088e81fdca6a73a663a48901bb5245acc8

src/com/android/settings/fingerprint/FingerprintEnrollFindSensor.java

diff --git a/src/com/android/settings/fingerprint/FingerprintEnrollFindSensor.java b/src/com/android/settings/fingerprint/FingerprintEnrollFindSensor.java
index b246bdf..a5af5cc 100644
--- a/src/com/android/settings/fingerprint/FingerprintEnrollFindSensor.java
+++ b/src/com/android/settings/fingerprint/FingerprintEnrollFindSensor.java
@@ -59,6 +59,12 @@
     @Override
     protected void onStop() {
         super.onStop();
+        mAnimation.pauseAnimation();
+    }
+
+    @Override
+    protected void onDestroy() {
+        super.onDestroy();
         mAnimation.stopAnimation();
     }
 

src/com/android/settings/fingerprint/FingerprintFindSensorAnimation.java

diff --git a/src/com/android/settings/fingerprint/FingerprintFindSensorAnimation.java b/src/com/android/settings/fingerprint/FingerprintFindSensorAnimation.java
index cb254ba..07ab99d 100644
--- a/src/com/android/settings/fingerprint/FingerprintFindSensorAnimation.java
+++ b/src/com/android/settings/fingerprint/FingerprintFindSensorAnimation.java
@@ -32,4 +32,9 @@
      */
     void stopAnimation();
 
+    /**
+     * Pause the animation
+     */
+    void pauseAnimation();
+
 }

src/com/android/settings/fingerprint/FingerprintLocationAnimationVideoView.java

diff --git a/src/com/android/settings/fingerprint/FingerprintLocationAnimationVideoView.java b/src/com/android/settings/fingerprint/FingerprintLocationAnimationVideoView.java
index c1fd2c2..5236272 100644
--- a/src/com/android/settings/fingerprint/FingerprintLocationAnimationVideoView.java
+++ b/src/com/android/settings/fingerprint/FingerprintLocationAnimationVideoView.java
@@ -117,11 +117,18 @@
 
     @Override
     public void stopAnimation() {
-        if (mMediaPlayer != null && mMediaPlayer.isPlaying()) {
+        if (mMediaPlayer != null) {
             mMediaPlayer.stop();
             mMediaPlayer.release();
             mMediaPlayer = null;
         }
     }
 
+    @Override
+    public void pauseAnimation() {
+        if (mMediaPlayer != null && mMediaPlayer.isPlaying()) {
+            mMediaPlayer.pause();
+        }
+    }
+
 }

src/com/android/settings/fingerprint/FingerprintLocationAnimationView.java

diff --git a/src/com/android/settings/fingerprint/FingerprintLocationAnimationView.java b/src/com/android/settings/fingerprint/FingerprintLocationAnimationView.java
index c17069f..cc1d4fd 100644
--- a/src/com/android/settings/fingerprint/FingerprintLocationAnimationView.java
+++ b/src/com/android/settings/fingerprint/FingerprintLocationAnimationView.java
@@ -112,6 +112,11 @@
         }
     }
 
+    @Override
+    public void pauseAnimation() {
+        stopAnimation();
+    }
+
     private void startPhase() {
         startRadiusAnimation();
         startAlphaAnimation();