commit cd6a1a9a6250c5c63e16d47b21dbaec56012a38f
author Tyler Gunn <tgunn@google.com> Thu Mar 29 13:48:29 2018 -0700
committer Tyler Gunn <tgunn@google.com> Fri Mar 30 17:03:43 2018 +0000
tree 436c5c26fc53b2e579b92cf62f97ef2b64dbcdcf
parent 4e4cbf37fcfb996d92ec161c86d336d89d18218e

Fix issue where EmergencyDialer is slow to open in some cases.

Per recommendations by framework team, using setShowWhenLocked instead of
the window flag.
In onPause, removing the ToneGenerator teardown.  This was a source of
long pauses during the activity lifecycle.  We now initialize it only in
onCreate and tear it down in onDestroy.

Bug: 76058395
Test: Manually reproduced user switching issue; confirmed it is fixed
Merged-In: I0bd63a9c7138ab9450861293cbd438105c33eb06
Change-Id: I0bd63a9c7138ab9450861293cbd438105c33eb06

src/com/android/phone/EmergencyDialer.java

diff --git a/src/com/android/phone/EmergencyDialer.java b/src/com/android/phone/EmergencyDialer.java
index 4cbfb38..09bb8bc 100644
--- a/src/com/android/phone/EmergencyDialer.java
+++ b/src/com/android/phone/EmergencyDialer.java
@@ -521,13 +521,6 @@
     @Override
     public void onPause() {
         super.onPause();
-
-        synchronized (mToneGeneratorLock) {
-            if (mToneGenerator != null) {
-                mToneGenerator.release();
-                mToneGenerator = null;
-            }
-        }
     }
 
     @Override
@@ -663,7 +656,7 @@
 
             // blur stuff behind the dialog
             dialog.getWindow().addFlags(WindowManager.LayoutParams.FLAG_BLUR_BEHIND);
-            dialog.getWindow().addFlags(WindowManager.LayoutParams.FLAG_SHOW_WHEN_LOCKED);
+            setShowWhenLocked(true);
         }
         return dialog;
     }