From afe44799f73b4140e25150156bf399e20211f509 Mon Sep 17 00:00:00 2001 From: Alison Cichowlas Date: Thu, 1 Jun 2017 16:07:32 -0400 Subject: [PATCH] Delay adding app settings link until data fetch returns, to avoid reordering/visible misplaced "additional settings in the app" during transition. Bug: 62051521 Test: manual with transition scale turned to 10x sloooow. Change-Id: I50666fbcb2a14344bbdbb2f4347c8470edb96619 --- .../android/settings/notification/AppNotificationSettings.java | 2 +- .../android/settings/notification/NotificationSettingsBase.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/com/android/settings/notification/AppNotificationSettings.java b/src/com/android/settings/notification/AppNotificationSettings.java index efcb5a166ee..6c4f27cd409 100644 --- a/src/com/android/settings/notification/AppNotificationSettings.java +++ b/src/com/android/settings/notification/AppNotificationSettings.java @@ -95,7 +95,6 @@ public class AppNotificationSettings extends NotificationSettingsBase { getPreferenceScreen().setOrderingAsAdded(true); setupBlock(); addHeaderPref(); - addAppLinkPref(); mShowLegacyChannelConfig = mBackend.onlyHasDefaultChannel(mAppRow.pkg, mAppRow.uid); if (mShowLegacyChannelConfig) { @@ -120,6 +119,7 @@ public class AppNotificationSettings extends NotificationSettingsBase { return; } populateChannelList(); + addAppLinkPref(); } }.execute(); } diff --git a/src/com/android/settings/notification/NotificationSettingsBase.java b/src/com/android/settings/notification/NotificationSettingsBase.java index bfa93557a62..d9faef8a85a 100644 --- a/src/com/android/settings/notification/NotificationSettingsBase.java +++ b/src/com/android/settings/notification/NotificationSettingsBase.java @@ -263,7 +263,7 @@ abstract public class NotificationSettingsBase extends SettingsPreferenceFragmen } protected void addAppLinkPref() { - if (mAppRow.settingsIntent != null) { + if (mAppRow.settingsIntent != null && mAppLink == null) { mAppLink = new Preference(getPrefContext()); mAppLink.setKey(KEY_APP_LINK); mAppLink.setOrder(500);