diff options
| author | yueg <yueg@google.com> | 2018-01-09 11:02:47 -0800 |
|---|---|---|
| committer | Copybara-Service <copybara-piper@google.com> | 2018-01-09 11:28:12 -0800 |
| commit | a3305355f1ee123bf981e9d47c82d6a11b5cb4b5 (patch) | |
| tree | b38dda77798db82df47b5a792cf58ec3ae2d6614 /java/com/android/incallui/InCallActivity.java | |
| parent | c07d6bee10c8d50c41d1513e6b8c8e9f85d3107e (diff) | |
| download | android_packages_apps_Dialer-a3305355f1ee123bf981e9d47c82d6a11b5cb4b5.tar.gz android_packages_apps_Dialer-a3305355f1ee123bf981e9d47c82d6a11b5cb4b5.tar.bz2 android_packages_apps_Dialer-a3305355f1ee123bf981e9d47c82d6a11b5cb4b5.zip | |
InCallActivity should not call onUiShowing(false) when recreating.
InCallActivity recreates when onNewIntent() and is not visible. Althought onPause() is called, we should not call onUiShowing(false) since the activity is going to be visible. The extra call confuses bubble animation.
Bug: 67605985,71746139
Test: InCallActivityTest
PiperOrigin-RevId: 181343690
Change-Id: I4d540be0e3e38400322f9b9c3e71e50b1f85977c
Diffstat (limited to 'java/com/android/incallui/InCallActivity.java')
| -rw-r--r-- | java/com/android/incallui/InCallActivity.java | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/java/com/android/incallui/InCallActivity.java b/java/com/android/incallui/InCallActivity.java index f09f0d944..0a5666690 100644 --- a/java/com/android/incallui/InCallActivity.java +++ b/java/com/android/incallui/InCallActivity.java @@ -479,7 +479,9 @@ public class InCallActivity extends TransactionSafeFragmentActivity dialpadFragment.onDialerKeyUp(null); } - InCallPresenter.getInstance().onUiShowing(false); + if (!isRecreating) { + InCallPresenter.getInstance().onUiShowing(false); + } if (isFinishing()) { InCallPresenter.getInstance().unsetActivity(this); } @@ -590,7 +592,8 @@ public class InCallActivity extends TransactionSafeFragmentActivity } } - private void onNewIntent(Intent intent, boolean isRecreating) { + @VisibleForTesting + void onNewIntent(Intent intent, boolean isRecreating) { this.isRecreating = isRecreating; // We're being re-launched with a new Intent. Since it's possible for a single InCallActivity |